neo.mjs 5.13.9 → 5.13.10

package/apps/ServiceWorker.mjs

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

package/examples/ServiceWorker.mjs

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

package/package.json

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

package/resources/scss/src/form/field/Text.scss

@@ -273,6 +273,7 @@
             border-color    : transparent;
             margin          : 0; // important for Safari => #1125
             padding-left    : 16px;
+            position        : relative; // overwriting the browser default static, to ensure the z-index gets applied
             z-index         : 1;
         }
 

package/src/DefaultConfig.mjs

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

package/src/component/Base.mjs

@@ -1596,7 +1596,7 @@ class Base extends CoreBase {
         }
 
         return new Promise((resolve, reject) => {
-            if (me.mounted) {
+            if (me.mounted && me.vnode) {
                 me.updateVdom(vdom, vnode, resolve, reject);
             } else {
                 me.update();
@@ -1837,22 +1837,24 @@ class Base extends CoreBase {
 
         // keep the vnode parent tree in sync
         ComponentManager.getParents(me).forEach((component, index) => {
-            if (!me.vnode) {
-                if (index === 0 && !VNodeUtil.removeChildVnode(component.vnode, me.id)) {
-                    // This can fail, in case the vnode is already removed (not an issue, better safe than sorry)
-                    // console.warn('syncVnodeTree: Could not remove the parent vnode for', me.id, component);
+            if (component.vnode) {
+                if (!me.vnode) {
+                    if (index === 0 && !VNodeUtil.removeChildVnode(component.vnode, me.id)) {
+                        // This can fail, in case the vnode is already removed (not an issue, better safe than sorry)
+                        // console.warn('syncVnodeTree: Could not remove the parent vnode for', me.id, component);
+                    }
                 }
-            }
 
-            // check for dynamically rendered components which get inserted into the component tree
-            else if (index === 0 && me.vnode.outerHTML) {
-                // console.log('dyn item', me.vnode, me.parentIndex);
-                component.vnode.childNodes.splice(me.parentIndex || 0, 0, me.vnode);
-            }
+                // check for dynamically rendered components which get inserted into the component tree
+                else if (index === 0 && me.vnode.outerHTML) {
+                    // console.log('dyn item', me.vnode, me.parentIndex);
+                    component.vnode.childNodes.splice(me.parentIndex || 0, 0, me.vnode);
+                }
 
-            else if (!VNodeUtil.replaceChildVnode(component.vnode, me.vnode.id, me.vnode)) {
-                // todo: can happen for dynamically inserted container items
-                // console.warn('syncVnodeTree: Could not replace the parent vnode for', me.vnode.id, component);
+                else if (!VNodeUtil.replaceChildVnode(component.vnode, me.vnode.id, me.vnode)) {
+                    // todo: can happen for dynamically inserted container items
+                    // console.warn('syncVnodeTree: Could not replace the parent vnode for', me.vnode.id, component);
+                }
             }
         });
 
@@ -2024,18 +2026,21 @@ class Base extends CoreBase {
 
                 reject?.()
             }).then(data => {
-                // console.log('Component vnode updated', data);
-                me.vnode          = data.vnode;
-                me.isVdomUpdating = false;
-
-                deltas = data.deltas;
-
-                if (!Neo.config.useVdomWorker && deltas.length > 0) {
-                    Neo.applyDeltas(me.appName, deltas).then(() => {
+                // checking if the component got destroyed before the update cycle is done
+                if (me.id) {
+                    // console.log('Component vnode updated', data);
+                    me.vnode          = data.vnode;
+                    me.isVdomUpdating = false;
+
+                    deltas = data.deltas;
+
+                    if (!Neo.config.useVdomWorker && deltas.length > 0) {
+                        Neo.applyDeltas(me.appName, deltas).then(() => {
+                            me.resolveVdomUpdate(resolve)
+                        });
+                    } else {
                         me.resolveVdomUpdate(resolve)
-                    });
-                } else {
-                    me.resolveVdomUpdate(resolve)
+                    }
                 }
             })
         }