neo.mjs 5.14.2 → 5.15.0

package/src/component/wrapper/GoogleMaps.mjs

@@ -35,10 +35,9 @@ class GoogleMaps extends Base {
                     type: 'Object'
                 }, {
                     name: 'icon',
-                    type: 'String'
+                    type: 'Object'
                 }, {
-                    name: 'id',
-                    type: 'String'
+                    name: 'id'
                 }, {
                     name: 'label',
                     type: 'String'
@@ -70,27 +69,27 @@ class GoogleMaps extends Base {
      * Internal flag. Gets set to true once Neo.main.addon.GoogleMaps.create() is finished.
      * @member {Boolean} mapCreated=false
      */
-    mapCreated        = false
+    mapCreated = false
     /**
      * Pass any options to the map instance which are not explicitly defined here
      * @member {Object} mapOptions={}
      */
-    mapOptions        = {}
+    mapOptions = {}
     /**
      * null => the maximum zoom from the current map type is used instead
      * @member {Number|null} maxZoom=null
      */
-    maxZoom           = null
+    maxZoom = null
     /**
      null => the minimum zoom from the current map type is used instead
      * @member {Number|null} minZoom=null
      */
-    minZoom           = null
+    minZoom = null
     /**
      * false hides the default zoom control
      * @member {Boolean} zoomControl=true
      */
-    zoomControl       = true
+    zoomControl = true
 
     /**
      * @param {Object} config
@@ -215,10 +214,7 @@ class GoogleMaps extends Base {
                 value
             });
 
-            me.fire('zoomChange', {
-                id: me.id,
-                value
-            })
+            me.fire('zoomChange', {id: me.id, value})
         }
     }
 
@@ -267,13 +263,6 @@ class GoogleMaps extends Base {
         this.zoom = data.value
     }
 
-    /**
-     * @param {Object} record
-     */
-    onMarkerClick(record) {
-        console.log('onMarkerClick', record)
-    }
-
     /**
      *
      */
@@ -317,12 +306,9 @@ class GoogleMaps extends Base {
 
         data.record = me.markerStore.get(data.id);
 
-        me.onMarkerClick(data);
+        me.onMarkerClick?.(data);
 
-        me.fire('markerClick', {
-            id: me.id,
-            data
-        })
+        me.fire('markerClick', {id: me.id, data})
     }
 
     /**

package/src/container/Base.mjs

@@ -152,6 +152,26 @@ class Base extends Component {
         }
     }
 
+    /**
+     * Triggered after the needsVdomUpdate config got changed
+     * @param {Boolean} value
+     * @param {Boolean} oldValue
+     * @protected
+     */
+    afterSetNeedsVdomUpdate(value, oldValue) {
+        if (!value) {
+            this.items?.forEach(item => {
+                // check for e.g. Toolbar items like '->'
+                if (typeof item !== 'string') {
+                    // we can not set the config directly => it could already be false,
+                    // and we still want to pass it further into subtrees
+                    item._needsVdomUpdate = false;
+                    item.afterSetNeedsVdomUpdate?.(false, true)
+                }
+            })
+        }
+    }
+
     /**
      * Triggered after the rendering config got changed
      * @param {Boolean} value

package/src/form/field/FileUpload.mjs

@@ -0,0 +1,34 @@
+import Base from '../../form/field/Base.mjs';
+
+/**
+ * @class Neo.form.field.FileUpload
+ * @extends Neo.form.field.Base
+ */
+class FileUpload extends Base {
+    static config = {
+        /**
+         * @member {String} className='Neo.form.field.FileUpload'
+         * @protected
+         */
+        className: 'Neo.form.field.FileUpload',
+        /**
+         * @member {String} ntype='file-upload-field'
+         * @protected
+         */
+        ntype: 'file-upload-field',
+        /**
+         * @member {String[]}} baseCls=['neo-file-upload-field']
+         * @protected
+         */
+        baseCls: ['neo-file-upload-field'],
+        /**
+         * @member {Object}} _vdom
+         */
+        _vdom:
+        {tag: 'input', type: 'file'}
+    }
+}
+
+Neo.applyClassConfig(FileUpload);
+
+export default FileUpload;

package/src/menu/List.mjs

@@ -342,14 +342,7 @@ class List extends BaseList {
     onItemClick(node, data) {
         super.onItemClick(node, data);
 
-        let me     = this,
-            record = data.record;
-
-        record.handler?.call(me, record);
-
-        if (me.hideOnLeafItemClick && !record.items) {
-            me.unmount()
-        }
+        this.onKeyDownEnter(node.id)
     }
 
     /**
@@ -369,7 +362,7 @@ class List extends BaseList {
             }
 
             if (record.items) {
-                submenu = me.subMenuMap[me.getMenuMapId(recordId)];
+                submenu = me.subMenuMap?.[me.getMenuMapId(recordId)];
 
                 if (submenu) {
                     me.toggleSubMenu(nodeId, record)
@@ -456,9 +449,9 @@ class List extends BaseList {
     toggleSubMenu(nodeId, record) {
         let me       = this,
             recordId = record[me.getKeyProperty()],
-            submenu  = me.subMenuMap[me.getMenuMapId(recordId)];
+            submenu  = me.subMenuMap?.[me.getMenuMapId(recordId)];
 
-        if (!submenu?.mounted) {
+        if (!submenu || !submenu.mounted) {
             me.showSubMenu(nodeId, record)
         } else {
             me.hideSubMenu()

package/src/selection/ListModel.mjs

@@ -102,7 +102,7 @@ class ListModel extends Model {
                 } else {
                     preventSelection = true;
                     me.deselectAll();
-                    view.fire('selectPreFirstItem');
+                    view.fire('selectPreFirstItem')
                 }
             } else if (index >= maxItems) {
                 if (me.stayInList) {
@@ -114,11 +114,11 @@ class ListModel extends Model {
                 } else {
                     preventSelection = true;
                     me.deselectAll();
-                    view.fire('selectPostLastItem');
+                    view.fire('selectPostLastItem')
                 }
             }
         } else {
-            index = 0;
+            index = 0
         }
 
         if (!preventSelection) {
@@ -127,7 +127,7 @@ class ListModel extends Model {
 
             me.select(itemId);
             view.focus(itemId);
-            view.fire('itemNavigate', record);
+            view.fire('itemNavigate', record)
         }
     }
 
@@ -148,7 +148,7 @@ class ListModel extends Model {
             {fn: 'onKeyDownLeft'   ,key: 'Left'   ,scope: id},
             {fn: 'onKeyDownRight'  ,key: 'Right'  ,scope: id},
             {fn: 'onKeyDownUp'     ,key: 'Up'     ,scope: id}
-        );
+        )
     }
 
     /**
@@ -161,7 +161,7 @@ class ListModel extends Model {
 
         if (itemId) {
             this.select(itemId);
-            view.focus(itemId);
+            view.focus(itemId)
         }
     }
 
@@ -182,7 +182,7 @@ class ListModel extends Model {
             {fn: 'onKeyDownUp'     ,key: 'Up'     ,scope: id}
         ]);
 
-        super.unregister();
+        super.unregister()
     }
 }
 

package/src/selection/TreeModel.mjs

@@ -0,0 +1,39 @@
+import ListModel from './ListModel.mjs';
+
+/**
+ * @class Neo.selection.TreeModel
+ * @extends Neo.selection.ListModel
+ */
+class TreeModel extends ListModel {
+    static config = {
+        /**
+         * @member {String} className='Neo.selection.TreeModel'
+         * @protected
+         */
+        className: 'Neo.selection.TreeModel',
+        /**
+         * @member {String} ntype='selection-treemodel'
+         * @protected
+         */
+        ntype: 'selection-treemodel'
+    }
+
+    /**
+     * @param {Object} data
+     */
+    onKeyDownEnter(data) {
+        console.log('onKeyDownEnter', data)
+    }
+
+    /**
+     * @param {Object} data
+     * @param {Number} step
+     */
+    onNavKey(data, step) {
+        console.log('onNavKey', data, step)
+    }
+}
+
+Neo.applyClassConfig(TreeModel);
+
+export default TreeModel;

package/src/tree/List.mjs

@@ -1,6 +1,8 @@
-import Base       from '../list/Base.mjs';
-import Collection from '../collection/Base.mjs';
-import NeoArray   from '../util/Array.mjs';
+import Base            from '../list/Base.mjs';
+import ClassSystemUtil from '../util/ClassSystem.mjs';
+import Collection      from '../collection/Base.mjs';
+import NeoArray        from '../util/Array.mjs';
+import TreeModel       from '../selection/TreeModel.mjs';
 
 /**
  * @class Neo.tree.List
@@ -23,10 +25,9 @@ class Tree extends Base {
          */
         baseCls: ['neo-tree-list'],
         /**
-         * todo: change the default to false once selection.TreeList is in place
-         * @member {Boolean} disableSelection=true
+         * @member {Boolean} disableSelection=false
          */
-        disableSelection: true,
+        disableSelection: false,
         /**
          * @member {Neo.draggable.tree.DragZone|null} dragZone=null
          */
@@ -56,7 +57,7 @@ class Tree extends Base {
          */
         _vdom:
         {cn: [
-            {tag: 'ul', cls: ['neo-list-container', 'neo-list'], cn: []}
+            {tag: 'ul', cls: ['neo-list-container', 'neo-list'], tabIndex: -1, cn: []}
         ]}
     }
 
@@ -111,6 +112,18 @@ class Tree extends Base {
         }
     }
 
+    /**
+     * Triggered before the selectionModel config gets changed.
+     * @param {Neo.selection.Model} value
+     * @param {Neo.selection.Model} oldValue
+     * @returns {Neo.selection.Model}
+     * @protected
+     */
+    beforeSetSelectionModel(value, oldValue) {
+        oldValue?.destroy();
+        return ClassSystemUtil.beforeSetInstance(value, TreeModel);
+    }
+
     /**
      * Triggered before the store config gets changed.
      * @param {Object|Neo.data.Store} value