neo.mjs 8.1.3 → 8.3.0

package/apps/ServiceWorker.mjs

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

package/apps/portal/index.html

@@ -16,7 +16,7 @@
         "@type": "Organization",
         "name": "Neo.mjs"
       },
-      "datePublished": "2025-01-12",
+      "datePublished": "2025-01-13",
       "publisher": {
         "@type": "Organization",
         "name": "Neo.mjs"

package/apps/portal/view/home/FooterContainer.mjs

@@ -107,7 +107,7 @@ class FooterContainer extends Container {
             }, {
                 module: Component,
                 cls   : ['neo-version'],
-                html  : 'v8.1.3'
+                html  : 'v8.3.0'
             }]
         }],
         /**

package/examples/ServiceWorker.mjs

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

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "neo.mjs",
-    "version": "8.1.3",
+    "version": "8.3.0",
     "description": "The webworkers driven UI framework",
     "type": "module",
     "repository": {
@@ -54,7 +54,7 @@
         "envinfo": "^7.14.0",
         "fs-extra": "^11.2.0",
         "highlightjs-line-numbers.js": "^2.9.0",
-        "inquirer": "^12.3.0",
+        "inquirer": "^12.3.1",
         "marked": "^15.0.6",
         "monaco-editor": "0.50.0",
         "neo-jsdoc": "1.0.1",

package/src/DefaultConfig.mjs

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

package/src/component/Base.mjs

@@ -16,6 +16,7 @@ const
     addUnits            = value => value == null ? value : isNaN(value) ? value : `${value}px`,
     closestController   = Symbol.for('closestController'),
     closestProvider     = Symbol.for('closestProvider'),
+    {currentWorker}     = Neo,
     lengthRE            = /^\d+\w+$/,
     twoWayBindingSymbol = Symbol.for('twoWayBinding');
 
@@ -573,7 +574,7 @@ class Component extends Base {
     afterSetConfig(key, value, oldValue) {
         let me = this;
 
-        if (Neo.currentWorker.isUsingStateProviders && me[twoWayBindingSymbol] && oldValue !== undefined) {
+        if (currentWorker.isUsingStateProviders && me[twoWayBindingSymbol] && oldValue !== undefined) {
             let binding = me.bind?.[key];
 
             if (binding?.twoWay) {
@@ -1057,7 +1058,7 @@ class Component extends Base {
             controller = me.controller;
 
         if (value) {
-            Neo.currentWorker.insertThemeFiles(value, me.__proto__);
+            currentWorker.insertThemeFiles(value, me.__proto__);
 
             if (controller) {
                 controller.windowId = value
@@ -1544,7 +1545,7 @@ class Component extends Base {
             opts = {},
             deltas;
 
-        if (Neo.currentWorker.isSharedWorker) {
+        if (currentWorker.isSharedWorker) {
             opts.appName  = me.appName;
             opts.windowId = me.windowId
         }
@@ -1783,7 +1784,7 @@ class Component extends Base {
      * @returns {Neo.state.Provider|null}
      */
     getStateProvider(ntype) {
-        if (!Neo.currentWorker.isUsingStateProviders) {
+        if (!currentWorker.isUsingStateProviders) {
             return null
         }
 
@@ -2055,7 +2056,7 @@ class Component extends Base {
 
             me.render(true)
         } else {
-            await Neo.currentWorker.promiseMessage('main', {
+            await currentWorker.promiseMessage('main', {
                 action     : 'mountDom',
                 appName    : me.appName,
                 id         : me.id,
@@ -2282,10 +2283,19 @@ class Component extends Base {
      * @param {Boolean} [mount] Mount the DOM after the vnode got created
      */
     async render(mount) {
-        let me            = this,
-            autoMount     = mount || me.autoMount,
-            app           = me.app,
-            useVdomWorker = Neo.config.useVdomWorker;
+        let me              = this,
+            autoMount       = mount || me.autoMount,
+            {app}           = me,
+            {useVdomWorker} = Neo.config;
+
+        // Verify that the critical rendering path => CSS files for the new tree is in place
+        if (currentWorker.countLoadingThemeFiles !== 0) {
+            currentWorker.on('themeFilesLoaded', function() {
+                me.render(mount)
+            }, me, {once: true});
+
+            return
+        }
 
         me.rendering = true;
 
@@ -2612,11 +2622,11 @@ class Component extends Base {
                     deltas: [{id, style: delta}]
                 };
 
-                if (Neo.currentWorker.isSharedWorker) {
+                if (currentWorker.isSharedWorker) {
                     opts.appName = me.appName
                 }
 
-                Neo.currentWorker.sendMessage('main', opts)
+                currentWorker.sendMessage('main', opts)
             }
         }
     }
@@ -2659,7 +2669,14 @@ class Component extends Base {
                     && mounted
                     && vnode
                 ) {
-                    me.#executeVdomUpdate(vdom, vnode, resolve, reject)
+                    // Verify that the critical rendering path => CSS files for the new tree is in place
+                    if (currentWorker.countLoadingThemeFiles !== 0) {
+                        currentWorker.on('themeFilesLoaded', function() {
+                            me.updateVdom(vdom, vnode, resolve, reject)
+                        }, me, {once: true})
+                    } else {
+                        me.#executeVdomUpdate(vdom, vnode, resolve, reject)
+                    }
                 }
             }
         }

package/src/main/addon/Stylesheet.mjs

@@ -92,11 +92,12 @@ class Stylesheet extends Base {
      * @param {String} data.className
      * @param {String[]} data.folders
      */
-    addThemeFiles(data) {
+    async addThemeFiles(data) {
         let {className} = data,
             {config}    = Neo,
             env         = config.environment,
             path        = env.startsWith('dist/') ? '' : config.appPath.includes('docs') ? `../dist/${env}/` : `../../dist/${env}/`,
+            promises    = [],
             rootPath    = config.basePath.substring(6);
 
         if (className.startsWith('Neo.')) {
@@ -107,13 +108,15 @@ class Stylesheet extends Base {
 
         data.folders.forEach(folder => {
             if (folder === 'src' || folder.includes('theme-') && config.themes.includes(`neo-${folder}`)) {
-                this.createStyleSheet(
+                promises.push(this.createStyleSheet(
                     null,
                     null,
                     `${rootPath}${path}css/${folder}/${className}.css`
-                )
+                ))
             }
-        })
+        });
+
+        await Promise.all(promises)
     }
 
     /**
@@ -121,28 +124,34 @@ class Stylesheet extends Base {
      * @param {String} [name]
      * @param {String} [id]
      * @param {String} [href]
+     * @returns {Promise<void>}
      */
-    createStyleSheet(name, id, href) {
+    async createStyleSheet(name, id, href) {
         if (!name && !href) {
             throw new Error('createStyleSheet: you need to either pass a name or a href')
         }
 
-        let link = document.createElement('link'),
-            env  = Neo.config.environment,
-            path = env.startsWith('dist/') ? env : ('dist/' + env),
-            url  = href ? href : Neo.config.basePath + path + '/' + name;
+        return new Promise((resolve, reject) => {
+            let link = document.createElement('link'),
+                env  = Neo.config.environment,
+                path = env.startsWith('dist/') ? env : ('dist/' + env),
+                url  = href ? href : Neo.config.basePath + path + '/' + name;
 
-        Object.assign(link, {
-            href: url,
-            rel : 'stylesheet',
-            type: 'text/css'
-        });
+            Object.assign(link, {
+                href: url,
+                rel : 'stylesheet',
+                type: 'text/css'
+            });
 
-        if (id) {
-            link.id = id
-        }
+            if (id) {
+                link.id = id
+            }
 
-        document.head.appendChild(link)
+            link.addEventListener('error', function() {reject()})
+            link.addEventListener('load',  function() {resolve()})
+
+            document.head.appendChild(link)
+        })
     }
 
     /**

package/src/worker/App.mjs

@@ -20,6 +20,10 @@ class App extends Base {
          * @protected
          */
         className: 'Neo.worker.App',
+        /**
+         * @member {Number} countLoadingThemeFiles_=0
+         */
+        countLoadingThemeFiles_: 0,
         /**
          * Remote method access for other workers
          * @member {Object} remote
@@ -77,6 +81,18 @@ class App extends Base {
         Neo.setCssVariable = me.setCssVariable.bind(me)
     }
 
+    /**
+     * Triggered after the countLoadingThemeFiles config got changed
+     * @param {Number} value
+     * @param {Number} oldValue
+     * @protected
+     */
+    afterSetCountLoadingThemeFiles(value, oldValue) {
+        if (value === 0 && oldValue !== undefined) {
+            this.fire('themeFilesLoaded')
+        }
+    }
+
     /**
      * @param {String} appName
      * @param {Array|Object} deltas
@@ -319,10 +335,14 @@ class App extends Base {
 
                     ns[fileName] = true;
 
+                    me.countLoadingThemeFiles++;
+
                     Neo.main.addon.Stylesheet.addThemeFiles({
                         className: mapClassName || className,
                         folders  : themeFolders,
                         windowId
+                    }).then(() => {
+                        me.countLoadingThemeFiles--
                     })
                 }
             }