neo.mjs 4.0.38 → 4.0.39

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "neo.mjs",
-    "version": "4.0.38",
+    "version": "4.0.39",
     "description": "The webworkers driven UI framework",
     "type": "module",
     "repository": {

package/src/container/Base.mjs

@@ -254,13 +254,13 @@ class Base extends Component {
      * @protected
      */
     createItems() {
-        let me       = this,
-            items    = me._items,
-            layout   = me.layout,
-            vdom     = me.vdom,
-            vdomRoot = me.getVdomRoot();
+        let me        = this,
+            items     = me._items,
+            itemsRoot = me.getVdomItemsRoot(),
+            layout    = me.layout,
+            vdom      = me.vdom;
 
-        vdomRoot.cn = [];
+        itemsRoot.cn = [];
 
         items.forEach((item, index) => {
             items[index] = item = me.createItem(item, index);
@@ -269,7 +269,7 @@ class Base extends Component {
                 layout.applyChildAttributes(item, index);
             }
 
-            vdomRoot.cn.push(item.vdom);
+            itemsRoot.cn.push(item.vdom);
         });
 
         me.vdom = vdom;
@@ -316,7 +316,7 @@ class Base extends Component {
      * @returns {Object} The new vdom items root
      */
     getVdomItemsRoot() {
-        return this.vdom.cn;
+        return this.getVdomRoot();
     }
 
     /**
@@ -377,7 +377,7 @@ class Base extends Component {
 
             me.items = items;
 
-            vdom.cn.splice(index, 0, item.vdom);
+            me.getVdomItemsRoot().cn.splice(index, 0, item.vdom);
         }
 
         if (silent) {
@@ -385,8 +385,8 @@ class Base extends Component {
         } else {
             me.promiseVdomUpdate().then(() => {
                 me.fire('insert', {
-                    index: index,
-                    item : item
+                    index,
+                    item
                 });
             });
         }
@@ -427,9 +427,7 @@ class Base extends Component {
         let me   = this,
             item = me.items[fromIndex];
 
-        if (fromIndex !== toIndex) {
-            me.switchItems(toIndex, fromIndex);
-        }
+        fromIndex !== toIndex && me.switchItems(toIndex, fromIndex);
 
         return item;
     }
@@ -502,20 +500,16 @@ class Base extends Component {
         let me    = this,
             items = me.items,
             vdom  = me.vdom,
-            cn, item;
+            item;
 
         if (index >= items.length) {
             Neo.warn('Container.removeAt: index >= items.length. ' + me.id);
         } else {
             item = items[index];
 
-            // console.log('remove item', item.id);
-
             items.splice(index, 1);
 
-            cn = vdom.cn || vdom.childNodes || vdom.children;
-
-            cn.splice(index, 1);
+            me.getVdomItemsRoot().cn.splice(index, 1);
 
             me[silent && !destroyItem ? '_vdom' : 'vdom'] = vdom;
 
@@ -548,8 +542,8 @@ class Base extends Component {
             item2Index = Neo.isNumber(item2id) ? item2id : me.indexOf(item2id),
             vdom       = me.vdom;
 
-        NeoArray.move(me.items,              item2Index, item1Index);
-        NeoArray.move(me.getVdomItemsRoot(), item2Index, item1Index);
+        NeoArray.move(me.items,                 item2Index, item1Index);
+        NeoArray.move(me.getVdomItemsRoot().cn, item2Index, item1Index);
 
         me.vdom = vdom;
     }

package/src/container/Panel.mjs

@@ -132,7 +132,7 @@ class Panel extends Container {
                 config = {
                     ntype       : 'container',
                     flex        : 1,
-                    items       : items,
+                    items,
                     itemDefaults: me.itemDefaults,
                     ...containerConfig
                 };
@@ -155,7 +155,7 @@ class Panel extends Container {
                 config = {
                     ntype       : 'container',
                     flex        : 1,
-                    items       : items,
+                    items,
                     itemDefaults: me.itemDefaults,
                     ...containerConfig
                 };

package/src/form/Container.mjs

@@ -24,7 +24,7 @@ class Container extends BaseContainer {
          */
         cls: ['neo-form-container'],
         /**
-         * @member {Object} vdom={tag: 'form',cn: [],onsubmit:'return false;'}
+         * @member {Object} vdom={tag: 'form',cn:[],onsubmit:'return false;'}
          */
         vdom:
         {tag: 'form', cn: [], onsubmit: 'return false;'}
@@ -57,9 +57,7 @@ class Container extends BaseContainer {
             fields   = [];
 
         children.forEach(item => {
-            if (item instanceof BaseField) {
-                fields.push(item);
-            }
+            item instanceof BaseField && fields.push(item);
         });
 
         return fields;

package/src/form/Fieldset.mjs

@@ -189,9 +189,7 @@ class Fieldset extends Container {
     onConstructed() {
         super.onConstructed();
 
-        if (this.collapsed) {
-            this.afterSetCollapsed(true, false);
-        }
+        this.collapsed && this.afterSetCollapsed(true, false);
     }
 
     /**
@@ -221,16 +219,16 @@ class Fieldset extends Container {
         } else {
             if (me.legend) {
                 me.legend.setSilent({
-                    iconCls: iconCls,
-                    text   : title
+                    iconCls,
+                    text: title
                 });
 
                 delete me.legend.vdom.reomveDom;
             } else {
                 me.legend = me.insert(0, {
-                    module : Legend,
-                    iconCls: iconCls,
-                    text   : title,
+                    module: Legend,
+                    iconCls,
+                    text  : title,
                     ...me.legendConfig
                 });
             }

package/src/table/Container.mjs

@@ -357,6 +357,13 @@ class Container extends BaseContainer {
         return this.vdom.cn[0];
     }
 
+    /**
+     * @returns {Object[]} The new vdom items root
+     */
+    getVdomItemsRoot() {
+        return this.vdom.cn[0];
+    }
+
     /**
      * @returns {Neo.table.View}
      */

package/src/table/header/Toolbar.mjs

@@ -128,14 +128,6 @@ class Toolbar extends BaseToolbar {
         return 'base';
     }
 
-    /**
-     * Specify a different vdom items root if needed (useful in case this container uses a wrapper node).
-     * @returns {Object} The new vdom items root
-     */
-    getVdomItemsRoot() {
-        return this.vdom.cn[0].cn;
-    }
-
     /**
      * Specify a different vdom root if needed to apply the top level style attributes on a different level.
      * Make sure to use getVnodeRoot() as well, to keep the vdom & vnode trees in sync.