[BUG #8514] added disconnectFromWidget method to the layouts

qooxdoo · Aug 22, 2014 · cc69004 · cc69004
1 parent fcb48f8
commit cc69004
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/framework/source/class/qx/ui/mobile/core/Widget.js b/framework/source/class/qx/ui/mobile/core/Widget.js
@@ -484,8 +484,7 @@ qx.Bootstrap.define("qx.ui.mobile.core.Widget",
      * @param layout {qx.ui.mobile.layout.Abstract} The new layout or
      *     <code>null</code> to reset the layout.
      */
-    _setLayout : function(layout)
-    {
+    _setLayout : function(layout) {
       if (qx.core.Environment.get("qx.debug")) {
         if (layout) {
           qx.core.Assert.assertInstance(layout, qx.ui.mobile.layout.Abstract);
@@ -494,6 +493,11 @@ qx.Bootstrap.define("qx.ui.mobile.core.Widget",
 
       if (layout) {
         layout.connectToWidget(this);
+      } else {
+        var oldLayout = this.getLayout();
+        if (oldLayout) {
+          oldLayout.disconnectFromWidget(this);
+        }
       }
 
       this.__layoutManager = layout;

diff --git a/framework/source/class/qx/ui/mobile/layout/Abstract.js b/framework/source/class/qx/ui/mobile/layout/Abstract.js
@@ -155,6 +155,21 @@ qx.Bootstrap.define("qx.ui.mobile.layout.Abstract",
     },
 
 
+    /**
+     * This method is called by the widget to disconnect the widget from
+     * the layout.
+     *
+     * @param widget {qx.ui.mobile.core.Widget} The widget to disconnect from
+     */
+    disconnectFromWidget : function(widget)
+    {
+      if (this._widget) {
+        this._widget.removeClasses(this._getCssClasses());
+        this._widget = null;
+      }
+    },
+
+
 
     /**
      * Connects the layout to a given child widget. Can be overridden in a concrete