#7076 WIP

Author: tobiu

apps/portal/view/ViewportController.mjs

@@ -319,8 +319,11 @@ class ViewportController extends Controller {
 
                 // enable "fast clicking" on main nav items => do not replace a cube layout with a new instance of cube
                 if (container.layout.ntype !== 'layout-cube') {
-                    container.layout = {ntype: 'cube', activeIndex, fitContainer: true, hideInactiveCardsOnDestroy: true};
-                    await me.timeout(200);
+                    container.setSilent({
+                        layout: {ntype: 'cube', activeIndex, fitContainer: true, hideInactiveCardsOnDestroy: true}
+                    });
+                    await container.promiseUpdate();
+                    await me.timeout(50);
                 }
 
                 container.layout.activeIndex = index;

src/Neo.mjs

@@ -304,7 +304,11 @@ Neo = globalThis.Neo = Object.assign({
      */
     createConfig(target, key, initialValue) {
         if (Neo.hasPropertySetter(target, key)) {
-            throw('Config ' + key + '_ (' + target.className + ') already has a set method, use beforeGet, beforeSet & afterSet instead')
+            throw(
+`Invalid config in ${target.className}: '${key}_'. The config '${key}' is already defined as reactive by a parent class.
+To override the default value, use '${key}' (without the underscore) in your static config.
+If you intended to create custom logic, use the 'beforeGet${Neo.capitalize(key)}()', 'beforeSet${Neo.capitalize(key)}()', and 'afterSet${Neo.capitalize(key)}()' hooks instead of redefining the config.`
+            )
         }
 
         const

src/component/Base.mjs

@@ -1899,33 +1899,33 @@ class Component extends Base {
      * @returns {Promise<*>}
      */
     set(values={}, silent=false) {
-        let me             = this,
-            needsRendering = values.hidden === false && values.hidden !== me.hidden;
-
-        me.silentVdomUpdate = true;
-
-        super.set(values);
+        const
+            me        = this,
+            wasHidden = me.hidden;
 
-        me.silentVdomUpdate = false;
+        me.setSilent(values);
 
-        if (silent || !me.needsVdomUpdate) {
-            return Promise.resolve()
-        } else {
-            if (needsRendering) {
+        if (!silent && me.needsVdomUpdate) {
+            if (wasHidden && !me.hidden) {
                 me.show();
                 return Promise.resolve()
             }
 
             return me.promiseUpdate()
         }
+
+        return Promise.resolve()
     }
 
     /**
-     * Convenience shortcut calling set() with the silent flag
+     * A silent version of set(), which does not trigger a vdom update at the end.
+     * Useful for batching multiple config changes.
      * @param {Object} values={}
      */
-    setSilent(values = {}) {
-        return this.set(values, true)
+    setSilent(values={}) {
+        this.silentVdomUpdate = true;
+        super.set(values);
+        this.silentVdomUpdate = false
     }
 
     /**

src/layout/Cube.mjs

@@ -116,7 +116,7 @@ class Cube extends Card {
 
         container.mounted && container.update();
 
-        me.timeout(100).then(() => {
+        me.timeout(50).then(() => {
             container.addCls('neo-animate')
         })
     }

src/vdom/Helper.mjs

@@ -54,7 +54,8 @@ class Helper extends Base {
             return deltas;
         }
 
-        let attributes, delta, value, keys, styles, add, remove;
+        let delta = {},
+            attributes, value, keys, styles, add, remove;
 
         if (vnode.vtype === 'text' && vnode.textContent !== oldVnode.textContent) {
             deltas.default.push({
@@ -79,7 +80,6 @@ class Helper extends Base {
             });
 
             keys.forEach(prop => {
-                delta = {};
                 value = vnode[prop];
 
                 switch (prop) {
@@ -145,12 +145,12 @@ class Helper extends Base {
                         }
                         break
                 }
+            });
 
-                if (Object.keys(delta).length > 0) {
-                    delta.id = vnode.id;
-                    deltas.default.push(delta)
-                }
-            })
+            if (Object.keys(delta).length > 0) {
+                delta.id = vnode.id;
+                deltas.default.push(delta)
+            }
         }
 
         return deltas
@@ -698,7 +698,7 @@ class Helper extends Base {
                 throw new Error('Neo.vdom.Helper: StringFromVnode is not loaded yet for updates!');
             }
         }
-
+console.log(opts.vdom.id);
         vnode  = me.createVnode(opts.vdom);
         deltas = me.createDeltas({oldVnode: opts.vnode, vnode});