neo.mjs 5.12.0 → 5.12.1

package/apps/ServiceWorker.mjs

@@ -20,9 +20,9 @@ class ServiceWorker extends ServiceBase {
          */
         singleton: true,
         /**
-         * @member {String} version='5.12.0'
+         * @member {String} version='5.12.1'
          */
-        version: '5.12.0'
+        version: '5.12.1'
     }
 
     /**

package/examples/ServiceWorker.mjs

@@ -20,9 +20,9 @@ class ServiceWorker extends ServiceBase {
          */
         singleton: true,
         /**
-         * @member {String} version='5.12.0'
+         * @member {String} version='5.12.1'
          */
-        version: '5.12.0'
+        version: '5.12.1'
     }
 
     /**

package/examples/button/base/MainContainer.mjs

@@ -170,6 +170,9 @@ class MainContainer extends ConfigurationViewport {
         }];
     }
 
+    /**
+     * @returns {Neo.component.Base}
+     */
     createExampleComponent() {
         return Neo.create({
             module   : Button,

package/examples/menu/list/MainContainer.mjs

@@ -45,7 +45,7 @@ class MainContainer extends ConfigurationViewport {
     createExampleComponent() {
         return Neo.create({
             module      : MenuList,
-            displayField: 'name',
+            displayField: 'text',
             store       : MainStore
         });
     }

package/examples/menu/list/MainStore.mjs

@@ -17,48 +17,48 @@ class MainStore extends Store {
         data: [{
             iconCls: 'fa fa-user',
             id     : 1,
-            name   : 'Item 1'
+            text   : 'Item 1'
         }, {
             iconCls: 'fa fa-home',
             id     : 2,
-            name   : 'Group 1',
+            text   : 'Group 1',
             items  : [{
                 iconCls: 'fa fa-home',
                 id     : 6,
-                name   : 'Item 1'
+                text   : 'Item 1'
             }, {
                 iconCls: 'fa fa-home',
                 id     : 7,
-                name   : 'Item 2'
+                text   : 'Item 2'
             }, {
                 iconCls: 'fa fa-home',
                 id     : 8,
-                name   : 'Item 3'
+                text   : 'Item 3'
             }]
         }, {
             iconCls: 'fa fa-cog',
             id     : 3,
-            name   : 'Item 2'
+            text   : 'Item 2'
         }, {
             iconCls: 'far fa-calendar',
             id     : 4,
-            name   : 'Item 3'
+            text   : 'Item 3'
         }, {
             iconCls: 'far fa-clock',
             id     : 5,
-            name   : 'Group 2',
+            text   : 'Group 2',
             items  : [{
                 iconCls: 'fa fa-clock',
                 id     : 9,
-                name   : 'Item 1'
+                text   : 'Item 1'
             }, {
                 iconCls: 'fa fa-clock',
                 id     : 10,
-                name   : 'Item 2'
+                text   : 'Item 2'
             }, {
                 iconCls: 'fa fa-clock',
                 id     : 11,
-                name   : 'Item 3'
+                text   : 'Item 3'
             }]
         }]
     }

package/examples/menu/panel/MainContainer.mjs

@@ -56,70 +56,70 @@ class MainContainer extends ConfigurationViewport {
             module: Menu,
 
             listConfig: {
-                displayField: 'name'
+                displayField: 'text'
             },
 
             listItems: [{
                 iconCls: 'fa fa-user',
                 id     : 1,
-                name   : 'Item 1'
+                text   : 'Item 1'
             }, {
                 iconCls: 'fa fa-home',
                 id     : 2,
-                name   : 'Group 1',
+                text   : 'Group 1',
                 items  : [{
                     iconCls: 'fa fa-home',
                     id     : 6,
-                    name   : 'Item 1'
+                    text   : 'Item 1'
                 }, {
                     iconCls: 'fa fa-home',
                     id     : 7,
-                    name   : 'Item 2'
+                    text   : 'Item 2'
                 }, {
                     iconCls: 'fa fa-home',
                     id     : 8,
-                    name   : 'Item 3'
+                    text   : 'Item 3'
                 }]
             }, {
                 iconCls: 'fa fa-cog',
                 id     : 3,
-                name   : 'Item 2'
+                text   : 'Item 2'
             }, {
                 iconCls: 'far fa-calendar',
                 id     : 4,
-                name   : 'Item 3'
+                text   : 'Item 3'
             }, {
                 iconCls: 'far fa-clock',
                 id     : 5,
-                name   : 'Group 2',
+                text   : 'Group 2',
                 items  : [{
                     iconCls: 'fa fa-clock',
                     id     : 9,
-                    name   : 'Item 1'
+                    text   : 'Item 1'
                 }, {
                     iconCls: 'fa fa-clock',
                     id     : 10,
-                    name   : 'Item 2'
+                    text   : 'Item 2'
                 }, {
                     iconCls: 'fa fa-clock',
                     id     : 11,
-                    name   : 'Group 1',
+                    text   : 'Group 1',
                     items  : [{
                         iconCls: 'far fa-clock',
                         id     : 12,
-                        name   : 'Item 1'
+                        text   : 'Item 1'
                     }, {
                         iconCls: 'far fa-clock',
                         id     : 13,
-                        name   : 'Item 2'
+                        text   : 'Item 2'
                     }, {
                         iconCls: 'far fa-clock',
                         id     : 14,
-                        name   : 'Item 3'
+                        text   : 'Item 3'
                     }]
                 }]
             }]
-        });
+        })
     }
 }
 

package/examples/table/container/MainContainer.mjs

@@ -79,9 +79,10 @@ class MainContainer extends ConfigurationViewport {
             valueLabelText: 'Cell & Column & Row'
         }, {
             module   : Checkbox,
+            checked  : me.exampleComponent.sortable,
             labelText: 'sortable',
             listeners: {change: me.onConfigChange.bind(me, 'sortable')},
-            value    : me.exampleComponent.sortable
+            style    : {marginTop: '10px'}
         }];
     }
 
@@ -90,7 +91,6 @@ class MainContainer extends ConfigurationViewport {
      */
     createExampleComponent() {
         return Neo.create(TableContainer, {
-            autoRender    : false,
             id            : 'myTableStoreContainer',
             selectionModel: CellModel,
             store         : MainStore,
@@ -101,6 +101,7 @@ class MainContainer extends ConfigurationViewport {
                 {dataField: 'githubId',  text: 'Github Id'},
                 {dataField: 'country',   text: 'Country'},
                 {
+                    text: 'Edit Action',
                     renderer: data => {
                         let button = Neo.create({
                             module  : Button,

package/package.json

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

package/src/DefaultConfig.mjs

@@ -236,12 +236,12 @@ const DefaultConfig = {
     useVdomWorker: true,
     /**
      * buildScripts/injectPackageVersion.mjs will update this value
-     * @default '5.12.0'
+     * @default '5.12.1'
      * @memberOf! module:Neo
      * @name config.version
      * @type String
      */
-    version: '5.12.0'
+    version: '5.12.1'
 };
 
 Object.assign(DefaultConfig, {

package/src/button/Base.mjs

@@ -186,6 +186,11 @@ class Base extends Component {
             click: value,
             scope: me.handlerScope || me
         });
+
+        me.menu && me.addDomListeners({
+            click: me.toggleMenu,
+            scope: me
+        })
     }
 
     /**
@@ -201,7 +206,7 @@ class Base extends Component {
         NeoArray.add(   iconNode.cls, value);
 
         iconNode.removeDom = !value || value === '';
-        this.update();
+        this.update()
     }
 
     /**
@@ -222,7 +227,7 @@ class Base extends Component {
         }
 
         iconNode.style.color = value;
-        this.update();
+        this.update()
     }
 
     /**
@@ -249,20 +254,51 @@ class Base extends Component {
     afterSetMenu(value, oldValue) {
         if (value) {
             import('../menu/List.mjs').then(module => {
-                let list = Neo.create({
-                    module  : module.default,
-                    appName : this.appName,
-                    floating: true,
-                    items   : value
+                let me = this;
+
+                me.menuList = Neo.create({
+                    module      : module.default,
+                    appName     : me.appName,
+                    displayField: 'text',
+                    floating    : true,
+                    hidden      : true,
+                    items       : value,
+                    parentId    : me.id
                 });
 
-                list.render(true);
-
-                console.log('afterSetMenu', list);
+                me.vdom.cn.push(me.menuList.vdom)
             })
         }
     }
 
+    /**
+     * Triggered after the mounted config got changed
+     * @param {Boolean} value
+     * @param {Boolean} oldValue
+     * @protected
+     */
+    afterSetMounted(value, oldValue) {
+        super.afterSetMounted(value, oldValue);
+
+        let me = this,
+            style;
+
+        if (value && me.menu) {
+            setTimeout(() => {
+                me.getDomRect().then(rect => {
+                    style = me.menuList.style || {};
+
+                    Object.assign(style, {
+                        right: 0,
+                        top  : rect.height + 'px'
+                    });
+
+                    me.menuList.style = style
+                })
+            }, 50)
+        }
+    }
+
     /**
      * Triggered after the pressed config got changed
      * @param {Boolean} value
@@ -288,7 +324,7 @@ class Base extends Component {
         value && me.addDomListeners({
             click: me.changeRoute,
             scope: me
-        });
+        })
     }
 
     /**
@@ -504,6 +540,23 @@ class Base extends Component {
             }
         }, rippleEffectDuration);
     }
+
+    /**
+     *
+     */
+    toggleMenu() {
+        let menuList = this.menuList,
+            hidden   = !menuList.hidden;
+
+        menuList.hidden = hidden;
+
+        if (!hidden) {
+            setTimeout(() => {
+                console.log('focus'); // todo: does not activate the key nav
+                menuList.focus()
+            }, 500)
+        }
+    }
 }
 
 Neo.applyClassConfig(Base);

package/src/component/Base.mjs

@@ -1106,14 +1106,11 @@ class Base extends CoreBase {
      * @param {String} id=this.id
      */
     focus(id=this.id) {
-        let me = this;
-
-        // remote method access
         Neo.main.DomAccess.focus({
-            id: id || me.id
+            id
         }).catch(err => {
-            console.log('Error attempting to receive focus for component', err, me);
-        });
+            console.log('Error attempting to receive focus for component', err, this);
+        })
     }
 
     /**
@@ -1191,7 +1188,7 @@ class Base extends CoreBase {
     getMountedParentId() {
         let parentId  = this.parentId,
             parent    = Neo.getComponent(parentId),
-            itemsRoot = parent?.getVdomItemsRoot();
+            itemsRoot = parent?.getVdomItemsRoot?.();
 
         return itemsRoot ? itemsRoot.id : parentId
     }

package/src/form/field/Text.mjs

@@ -1442,21 +1442,19 @@ class Text extends Base {
             }
         }
 
-        if (required && isEmpty) {
-            me._error = me.errorTextRequired;
-            returnValue = false;
-        } else if (Neo.isNumber(maxLength) && valueLength > maxLength) {
-            if (required || !isEmpty) {
-                me._error = me.errorTextMaxLength(errorParam);
+        if (isEmpty) {
+            if (required) {
+                me._error = me.errorTextRequired;
                 returnValue = false;
             }
-        } else if (Neo.isNumber(minLength) && valueLength < minLength) {
-            if (required || !isEmpty) {
+        } else {
+            if (Neo.isNumber(maxLength) && valueLength > maxLength) {
+                me._error = me.errorTextMaxLength(errorParam);
+                returnValue = false;
+            } else if (Neo.isNumber(minLength) && valueLength < minLength) {
                 me._error = me.errorTextMinLength(errorParam);
                 returnValue = false;
-            }
-        } else if (inputPattern && !inputPattern.test(value)) {
-            if (required || !isEmpty) {
+            } else if (inputPattern && !inputPattern.test(value)) {
                 me._error = me.errorTextInputPattern(errorParam);
                 returnValue = false;
             }

package/src/main/mixin/DeltaUpdates.mjs

@@ -127,12 +127,12 @@ class DeltaUpdates extends Base {
                 startTag  = `<!-- ${delta.id} -->`;
                 reg       = new RegExp(startTag + '[\\s\\S]*?<!-- \/neo-vtext -->');
 
-                node.innerHTML = node.innerHTML.replace(reg, '');
+                node.innerHTML = node.innerHTML.replace(reg, '')
             } else {
                 // console.warn('du_removeNode: dom node not found for id', delta.id);
             }
         } else {
-            node.parentNode.removeChild(node);
+            node.remove()
         }
     }
 

package/src/manager/DomEvent.mjs

@@ -106,7 +106,7 @@ class DomEvent extends Base {
 
                                 if (!preventFire) {
                                     // multiple listeners would change the reference of data.component
-                                    data = Neo.clone(data, true);
+                                    data = Neo.clone(data, true, true);
 
                                     data.component = component;
                                     listener.fn.apply(listener.scope || globalThis, [data]);

package/src/menu/List.mjs

@@ -96,7 +96,12 @@ class List extends BaseList {
          * We are applying a z-index style which is 1 number higher to each sub-menu
          * @member {Number} zIndex_=100
          */
-        zIndex_: 100
+        zIndex_: 100,
+        /**
+         * @member {Object} _vdom
+         */
+        _vdom:
+        {tag: 'ul', tabIndex: -1, cn: []}
     }
 
     /**
@@ -138,6 +143,7 @@ class List extends BaseList {
             if (me.isRoot) {
                 if (!value) {
                     me.focusTimeoutId = setTimeout(() => {
+                        console.log('unmount'); // todo: does not hide a top-level floating menu
                         me[me.floating ? 'unmount' : 'hideSubMenu']();
                     }, 20);
                 } else {
@@ -305,26 +311,27 @@ class List extends BaseList {
 
                 Object.assign(menuStyle, style);
 
-                subMenu.setSilent({style: menuStyle});
+                subMenu.setSilent({style: menuStyle})
             } else {
                 subMenuMap[subMenuMapId] = subMenu = Neo.create({
-                    module     : List,
-                    appName    : me.appName,
-                    floating   : true,
-                    items      : record.items,
-                    isRoot     : false,
-                    parentId   : Neo.apps[me.appName].mainView.id,
-                    parentIndex: store.indexOf(record),
-                    parentMenu : me,
+                    module      : List,
+                    appName     : me.appName,
+                    displayField: me.displayField,
+                    floating    : true,
+                    items       : record.items,
+                    isRoot      : false,
+                    parentId    : Neo.apps[me.appName].mainView.id,
+                    parentIndex : store.indexOf(record),
+                    parentMenu  : me,
                     style,
-                    zIndex     : me.zIndex + 1
-                });
+                    zIndex      : me.zIndex + 1
+                })
             }
 
             me.activeSubMenu = subMenu;
             me.subMenuMap    = subMenuMap;
 
-            subMenu.render(true);
+            subMenu.render(true)
         });
     }
 
@@ -335,7 +342,7 @@ class List extends BaseList {
         this.selectionModel.deselectAll(true); // silent update
         this.hideSubMenu();
 
-        super.unmount();
+        super.unmount()
     }
 }
 

package/src/menu/Model.mjs

@@ -28,7 +28,7 @@ class Model extends BaseModel {
             name: 'items', // optional
             type: 'Array'
         }, {
-            name: 'name',
+            name: 'text',
             type: 'String'
         }]
     }