neo.mjs 6.15.6 → 6.15.7

package/apps/ServiceWorker.mjs

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

package/examples/ServiceWorker.mjs

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

package/package.json

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

package/src/DefaultConfig.mjs

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

package/src/core/Util.mjs

@@ -178,6 +178,15 @@ class Util extends Base {
         return value?.constructor?.name === 'Object' || false
     }
 
+    /**
+     * Returns true if the passed value is a neo data record
+     * @param {Object} value The value to test
+     * @returns {Boolean}
+     */
+    static isRecord(value) {
+        return value?.constructor?.name === 'Record' || false
+    }
+
     /**
      * Returns true if the passed value is a string
      * @param {String} value The value to test
@@ -224,6 +233,7 @@ Neo.applyFromNs(Neo, Util, {
     isFunction       : 'isFunction',
     isNumber         : 'isNumber',
     isObject         : 'isObject',
+    isRecord         : 'isRecord',
     isString         : 'isString',
     toArray          : 'toArray'
 }, true);

package/src/form/field/ComboBox.mjs

@@ -306,7 +306,7 @@ class ComboBox extends Picker {
             record;
 
         // getting a record, nothing to do
-        if (Neo.isObject(value)) {
+        if (Neo.isRecord(value)) {
             return value
         }
 
@@ -392,7 +392,7 @@ class ComboBox extends Picker {
 
         // Filter resulting in something to show
         if (store.getCount()) {
-            me.getPicker().hidden = false;
+            me.showPicker();
 
             // List might not exist until the picker is created
             const
@@ -620,19 +620,6 @@ class ComboBox extends Picker {
         this.update()
     }
 
-    /**
-     *
-     */
-    onPickerTriggerClick() {
-        let me = this;
-
-        if (me.picker?.isVisible) {
-            me.picker.hidden = true
-        } else if (!me.disabled && !me.readOnly) {
-            me.doFilter(null)
-        }
-    }
-
     /**
      * Selecting a record, if required
      * @param {Object[]} items
@@ -679,6 +666,20 @@ class ComboBox extends Picker {
         me.list.selectItem(index)
     }
 
+    /**
+     * @override
+     */
+    togglePicker() {
+        let me       = this,
+            {picker} = me;
+
+        if (picker?.hidden === false) {
+            picker.hidden = true
+        } else if (!me.disabled && !me.readOnly) {
+            me.doFilter(null)
+        }
+    }
+
     /**
      * Override this method as needed inside class extensions.
      * @param {*} value
@@ -687,7 +688,7 @@ class ComboBox extends Picker {
     updateInputValueFromValue(value) {
         let inputValue = null;
 
-        if (Neo.isObject(value)) {
+        if (Neo.isRecord(value)) {
             inputValue = value[this.displayField]
         }
 

package/src/form/field/Picker.mjs

@@ -288,8 +288,8 @@ class Picker extends Text {
             // We processed this event, and it should not proceed to ancestor components
             data.cancelBubble = true;
 
-            // And no further listeers should be notified
-            return false;
+            // And no further listeners should be notified
+            return false
         }
     }