neo.mjs 4.0.41 → 4.0.44

package/buildScripts/watchThemes.mjs

@@ -20,7 +20,7 @@ fs.watch(scssPath, {
     recursive: true
 }, (eventType, filename) => {
     if (filename.endsWith('.scss')) {
-        switch (eventType) {
+        switch(eventType) {
             case 'change': {
                 buildFile(filename);
             }
@@ -45,51 +45,55 @@ function buildFile(filename) {
     ].join('');
 
     fs.readFile(filePath).then(content => {
-        let result = sass.renderSync({
-            data          : data + content.toString(),
-            outFile       : destPath,
-            sourceMap     : true,
-            sourceMapEmbed: false
-        });
+        try {
+            let result = sass.renderSync({
+                data          : data + content.toString(),
+                outFile       : destPath,
+                sourceMap     : true,
+                sourceMapEmbed: false
+            });
 
-        map = result.map?.toString();
+            map = result.map?.toString();
 
-        if (map) {
-            // https://github.com/neomjs/neo/issues/1970
-            map = JSON.parse(map);
+            if (map) {
+                // https://github.com/neomjs/neo/issues/1970
+                map = JSON.parse(map);
 
-            let filenameSlash = filename;
+                let filenameSlash = filename;
 
-            if (path.sep === '\\') {
-                filenameSlash = filenameSlash.replace(/\\/g, '/');
-            }
+                if (path.sep === '\\') {
+                    filenameSlash = filenameSlash.replace(/\\/g, '/');
+                }
 
-            let len = filenameSlash.split('/').length,
-                src = `/scss/${filenameSlash}`,
-                i   = 0;            
-            
-            for (; i < len; i++) {
-                src = '../' + src;
-            }            
+                let len = filenameSlash.split('/').length,
+                    src = `/scss/${filenameSlash}`,
+                    i   = 0;
 
-            map.sources = [src];
-        }
+                for (; i < len; i++) {
+                    src = '../' + src;
+                }
 
-        postcss([autoprefixer]).process(result.css, {
-            from: filePath,
-            to  : destPath,
-            map : {
-                prev: map && JSON.stringify(map)
+                map.sources = [src];
             }
-        }).then(result => {
-            fs.writeFileSync(destPath, result.css, () => true);
 
-            if (result.map) {
-                fs.writeFileSync(result.opts.to + '.map', result.map.toString());
-            }
-
-            const processTime = (Math.round((new Date - startDate) * 100) / 100000).toFixed(2);
-            console.log('Updated file:', (chalk.blue(`${processTime}s`)), destPath);
-        });
+            postcss([autoprefixer]).process(result.css, {
+                from: filePath,
+                to  : destPath,
+                map : {
+                    prev: map && JSON.stringify(map)
+                }
+            }).then(result => {
+                fs.writeFileSync(destPath, result.css, () => true);
+
+                if (result.map) {
+                    fs.writeFileSync(result.opts.to + '.map', result.map.toString());
+                }
+
+                const processTime = (Math.round((new Date - startDate) * 100) / 100000).toFixed(2);
+                console.log('Updated file:', (chalk.blue(`${processTime}s`)), destPath);
+            });
+        } catch(error) {
+            console.log('SCSS build failed for', chalk.red(filename));
+        }
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "neo.mjs",
-    "version": "4.0.41",
+    "version": "4.0.44",
     "description": "The webworkers driven UI framework",
     "type": "module",
     "repository": {
@@ -50,9 +50,9 @@
         "neo-jsdoc": "^1.0.1",
         "neo-jsdoc-x": "^1.0.4",
         "postcss": "^8.4.14",
-        "sass": "^1.52.2",
+        "sass": "^1.52.3",
         "webpack": "^5.73.0",
-        "webpack-cli": "^4.9.2",
+        "webpack-cli": "^4.10.0",
         "webpack-dev-server": "4.9.2",
         "webpack-hook-plugin": "^1.0.7",
         "webpack-node-externals": "^3.0.0"

package/src/component/Base.mjs

@@ -17,6 +17,13 @@ import VNodeUtil        from '../util/VNode.mjs';
  */
 class Base extends CoreBase {
     static getStaticConfig() {return {
+        /**
+         * Valid values for hideMode
+         * @member {String[]} hideModes=['removeDom','visibility']
+         * @protected
+         * @static
+         */
+        hideModes: ['removeDom', 'visibility'],
         /**
          * True automatically applies the core/Observable.mjs mixin
          * @member {Boolean} observable=true
@@ -147,12 +154,18 @@ class Base extends CoreBase {
          * @member {Number|String|null} height_=null
          */
         height_: null,
+        /**
+         * Initial setting to hide or show the component and
+         * you can use either hide()/show() or change this config directly to change the hidden state
+         * @member {Boolean} hidden_=false
+         */
+        hidden_: false,
         /**
          * Used for hide and show and defines if the component
          * should use css visibility:'hidden' or vdom:removeDom
-         * @member {'visible', 'remove'} hiddenType='visible'
+         * @member {String} hideMode_='visibility'
          */
-        hiddenType: 'visible',
+        hideMode_: 'visibility',
         /**
          * The top level innerHTML of the component
          * @member {String|null} html_=null
@@ -537,6 +550,18 @@ class Base extends CoreBase {
         this.changeVdomRootKey('height', value);
     }
 
+    /**
+     * Triggered after the hidden config got changed
+     * @param {Boolean} value
+     * @param {Boolean} oldValue
+     * @protected
+     */
+    afterSetHidden(value, oldValue) {
+        if (!(!value && oldValue === undefined)) {
+            this[value ? 'hide' : 'show']();
+        }
+    }
+
     /**
      * Triggered after the html config got changed
      * @param {String|null} value
@@ -742,6 +767,16 @@ class Base extends CoreBase {
         return value || [];
     }
 
+    /**
+     * Triggered before the hideMode config gets changed
+     * @param {String} value
+     * @param {String} oldValue
+     * @protected
+     */
+     beforeSetHideMode(value, oldValue) {
+        return this.beforeSetEnumValue(value, oldValue, 'hideMode');
+    }
+
     /**
      * Triggered before the keys config gets changed.
      * Creates a KeyNavigation instance if needed.
@@ -1080,14 +1115,14 @@ class Base extends CoreBase {
 
     /**
      * Hide the component.
-     * hiddenType: 'visible' uses css visibility.
-     * hiddenType: 'remove' uses vdom removeDom.
-     * If hiddenType 'remove' you can pass a timeout for custom css class hiding.
+     * hideMode: 'removeDom'  uses vdom removeDom.
+     * hideMode: 'visibility' uses css visibility.
+     * If hideMode === 'removeDom' you can pass a timeout for custom css class hiding.
      * @param {Number} timeout
      */
     hide(timeout) {
         let me       = this,
-            doRemove = me.hiddenType !== 'visible';
+            doRemove = me.hideMode !== 'visibility';
 
         if (doRemove) {
             let removeFn = function() {
@@ -1106,6 +1141,8 @@ class Base extends CoreBase {
             style.visibility = 'hidden';
             me.style = style;
         }
+
+        this._hidden = true;
     }
 
     /**
@@ -1464,12 +1501,12 @@ class Base extends CoreBase {
 
     /**
      * Show the component.
-     * hiddenType: 'visible' uses css visibility.
-     * hiddenType: 'remove' uses vdom removeDom.
+     * hideMode: 'removeDom'  uses vdom removeDom.
+     * hideMode: 'visibility' uses css visibility.
      */
     show() {
         let me    = this,
-            doAdd = me.hiddenType !== 'visible';
+            doAdd = me.hideMode !== 'visibility';
 
         if (doAdd) {
             let vdom = me.vdom;
@@ -1480,6 +1517,8 @@ class Base extends CoreBase {
             style.visibility = 'visible';
             me.style = style;
         }
+
+        this._hidden = false;
     }
 
     /**

package/src/data/RecordFactory.mjs

@@ -76,7 +76,7 @@ class RecordFactory extends Base {
 
                         if (Array.isArray(model.fields)) {
                             model.fields.forEach(field => {
-                                let parsedValue = instance.parseRecordValue(field, config[field.name]),
+                                let parsedValue = instance.parseRecordValue(field, config[field.name], config),
                                     symbol      = Symbol.for(field.name);
 
                                 properties = {
@@ -98,7 +98,7 @@ class RecordFactory extends Base {
                                                 oldValue = me[symbol];
 
                                             if (!Neo.isEqual(value, oldValue)) {
-                                                value = instance.parseRecordValue(field, value);
+                                                value = instance.parseRecordValue(field, value, null);
 
                                                 me[symbol] = value;
 
@@ -223,10 +223,22 @@ class RecordFactory extends Base {
      * todo: parse value for more field types
      * @param {Object} field
      * @param {*} value
+     * @param {Object} recordConfig
      * @returns {*}
      */
-    parseRecordValue(field, value) {
-        const type = field?.type.toLowerCase();
+    parseRecordValue(field, value, recordConfig) {
+        let mapping = field.mapping,
+            type    = field.type?.toLowerCase();
+
+        // only trigger mappings for initial values
+        // dynamic changes of a field will not pass the recordConfig
+        if (mapping && recordConfig) {
+            let ns  = mapping.split('.'),
+                key = ns.pop();
+
+            ns    = Neo.ns(ns, true, recordConfig);
+            value = ns[key];
+        }
 
         if (type === 'date') {
             return new Date(value);

package/src/form/field/trigger/Base.mjs

@@ -40,10 +40,6 @@ class Base extends Component {
          * @member {Neo.form.field.Base|null} field=null
          */
         field: null,
-        /**
-         * @member {Boolean} hidden_=false
-         */
-        hidden_: false,
         /**
          * @member {String|null} iconCls_=null
          */