container.Base: removeAt() without destroying the component instance …

…should remove layout related css rules #5438
neomjs · Jun 22, 2024 · b7bd601 · b7bd601
1 parent 19129e9
commit b7bd601
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/examples/component/helix/MainContainer.mjs b/examples/component/helix/MainContainer.mjs
@@ -62,7 +62,7 @@ class MainContainer extends Viewport {
             ntype    : 'panel',
             layout   : {ntype: 'vbox', align: 'stretch'},
             reference: 'controls-panel',
-            style    : {backgroundColor: '#2b2b2b', flex: 'none'},
+            style    : {backgroundColor: '#2b2b2b'},
             width    : 250,
 
             containerConfig: {

diff --git a/src/container/Base.mjs b/src/container/Base.mjs
@@ -678,6 +678,7 @@ class Base extends Component {
                 item.destroy(true, silent);
                 return null
             } else {
+                me.layout?.removeChildAttributes(item);
                 item.mounted = false;
                 return item
             }

diff --git a/src/layout/Flexbox.mjs b/src/layout/Flexbox.mjs
@@ -219,7 +219,7 @@ class Flexbox extends Base {
     }
 
     /**
-     * Removes all CSS rules from an container item this layout is bound to.
+     * Removes all CSS rules from a container item this layout is bound to.
      * Gets called when switching to a different layout.
      * @param {Neo.component.Base} item
      * @protected