From: Marc Englund <marc.englund@itmill.com>
Date: Mon, 12 Jan 2009 14:05:59 +0000 (+0000)
Subject: Remove window from application when closed. Fixes #2449
X-Git-Tag: 6.7.0.beta1~3357
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2244b9b137a4111b13dad534c7e65fe0c50250d8;p=vaadin-framework.git

Remove window from application when closed. Fixes #2449

svn changeset:6504/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/demo/sampler/SamplerApplication.java b/src/com/itmill/toolkit/demo/sampler/SamplerApplication.java
index 10dd8ebabe..18a5f11e2b 100644
--- a/src/com/itmill/toolkit/demo/sampler/SamplerApplication.java
+++ b/src/com/itmill/toolkit/demo/sampler/SamplerApplication.java
@@ -75,20 +75,11 @@ public class SamplerApplication extends Application {
     // Supports multiple browser windows
     @Override
     public Window getWindow(String name) {
-        /*- REST code, using fragments
-        if (features.getFeatureByPath(name) != null) {
-            return null;
-        }
-        -*/
         Window w = super.getWindow(name);
         if (w == null) {
             w = new SamplerWindow();
             w.setName(name);
             addWindow(w);
-            // secondary windows will support normal reload if this is
-            // enabled, but the url gets ugly:
-            // w.open(new ExternalResource(w.getURL()));
-
         }
         return w;
 
@@ -242,6 +233,14 @@ public class SamplerApplication extends Application {
             toggleBar.addComponent(mode);
             toggleBar.setComponentAlignment(mode, Alignment.MIDDLE_RIGHT);
 
+            addListener(new CloseListener() {
+                public void windowClose(CloseEvent e) {
+                    if (getMainWindow() != SamplerWindow.this) {
+                        SamplerApplication.this
+                                .removeWindow(SamplerWindow.this);
+                    }
+                }
+            });
         }
 
         /**
@@ -639,11 +638,15 @@ public class SamplerApplication extends Application {
                         grid.setRows(grid.getCursorY() + 1);
                         grid.addComponent(title, 0, grid.getCursorY(), grid
                                 .getColumns() - 1, grid.getCursorY());
-                        grid.setComponentAlignment(title, Alignment.MIDDLE_LEFT);
+                        grid
+                                .setComponentAlignment(title,
+                                        Alignment.MIDDLE_LEFT);
                     } else {
                         title.setStyleName("subsection");
                         grid.addComponent(title);
-                        grid.setComponentAlignment(title, Alignment.MIDDLE_LEFT);
+                        grid
+                                .setComponentAlignment(title,
+                                        Alignment.MIDDLE_LEFT);
                     }
 
                 } else {