Changeset 4802

Timestamp:

06/09/08 08:29:45 (6 months ago)

Author:

matti.tahvonen@…

Message:

fixes #1782

Files:

• trunk/src/com/itmill/toolkit/ui/Panel.java (modified) (1 diff)

trunk/src/com/itmill/toolkit/ui/Panel.java

@@ -109 +109 @@
 
     /**
-     * Sets the layout of the panel. All the components are moved to new layout.
-     * 
-     * @param layout
+     * Sets the layout of the panel.
+     * 
+     * If given layout is null, an OrderedLayout is used as a default.
+     * 
+     * Components from old layout are not moved to new layout by default
+     * (changed in 5.2.2). Use function in Layout interface manually.
+     * 
+     * @param newLayout
      *                the New layout of the panel.
      */
-    public void setLayout(Layout layout) {
+    public void setLayout(Layout newLayout) {
 
         // Only allow non-null layouts
-        if (layout == null) {
-            layout = new OrderedLayout();
+        if (newLayout == null) {
+            newLayout = new OrderedLayout();
             // Force margins by default
-            layout.setMargin(true);
-        }
-
-        if (layout == this.layout) {
+            newLayout.setMargin(true);
+        }
+
+        if (newLayout == layout) {
             // don't set the same layout twice
             return;
         }
 
+        // detach old layout if present
+        if (layout != null) {
+            layout.setParent(null);
+            layout
+                    .removeListener((ComponentContainer.ComponentAttachListener) this);
+            layout
+                    .removeListener((ComponentContainer.ComponentDetachListener) this);
+        }
+
         // Sets the panel to be parent for the layout
-        layout.setParent(this);
-
-        // If panel already contains a layout, move the contents to new one
-        // and detach old layout from the panel
-        if (this.layout != null) {
-            layout.moveComponentsFrom(this.layout);
-            this.layout.setParent(null);
-        }
-
-        // Removes the event listeners from the old layout
-        if (this.layout != null) {
-            this.layout
-                    .removeListener((ComponentContainer.ComponentAttachListener) this);
-            this.layout
-                    .removeListener((ComponentContainer.ComponentDetachListener) this);
-        }
+        newLayout.setParent(this);
 
         // Sets the new layout
-        this.layout = layout;
+        layout = newLayout;
 
         // Adds the event listeners for new layout
-        layout.addListener((ComponentContainer.ComponentAttachListener) this);
-        layout.addListener((ComponentContainer.ComponentDetachListener) this);
+        newLayout
+                .addListener((ComponentContainer.ComponentAttachListener) this);
+        newLayout
+                .addListener((ComponentContainer.ComponentDetachListener) this);
     }