neo.mjs 3.1.3 → 3.2.2

package/src/main/draggable/sensor/Base.mjs

@@ -75,4 +75,4 @@ class Base extends CoreBase {
 
 Neo.applyClassConfig(Base);
 
-export {Base as default};
+export default Base;

package/src/main/draggable/sensor/Mouse.mjs

@@ -213,4 +213,4 @@ function preventDefault(event) {
 
 Neo.applyClassConfig(Mouse);
 
-export {Mouse as default};
+export default Mouse;

package/src/main/draggable/sensor/Touch.mjs

@@ -242,4 +242,4 @@ function preventDefault(event) {
 
 Neo.applyClassConfig(Touch);
 
-export {Touch as default};
+export default Touch;

package/src/main/mixin/DeltaUpdates.mjs

@@ -297,4 +297,4 @@ class DeltaUpdates extends Base {
 
 Neo.applyClassConfig(DeltaUpdates);
 
-export {DeltaUpdates as default};
+export default DeltaUpdates;

package/src/main/mixin/TouchDomEvents.mjs

@@ -61,4 +61,4 @@ class TouchDomEvents extends Base {
 
 Neo.applyClassConfig(TouchDomEvents);
 
-export {TouchDomEvents as default};
+export default TouchDomEvents;

package/src/manager/Base.mjs

@@ -46,4 +46,4 @@ class Base extends CollectionBase{
 
 Neo.applyClassConfig(Base);
 
-export {Base as default};
+export default Base;

package/src/menu/List.mjs

@@ -341,4 +341,4 @@ class List extends BaseList {
 
 Neo.applyClassConfig(List);
 
-export {List as default};
+export default List;

package/src/menu/Model.mjs

@@ -36,4 +36,4 @@ class Model extends BaseModel {
 
 Neo.applyClassConfig(Model);
 
-export {Model as default};
+export default Model;

package/src/menu/Panel.mjs

@@ -89,4 +89,4 @@ class Panel extends BasePanel {
 
 Neo.applyClassConfig(Panel);
 
-export {Panel as default};
+export default Panel;

package/src/menu/Store.mjs

@@ -21,4 +21,4 @@ class Store extends BaseStore {
 
 Neo.applyClassConfig(Store);
 
-export {Store as default};
+export default Store;

package/src/model/Component.mjs

@@ -124,11 +124,9 @@ class Component extends Base {
      * @protected
      */
     beforeSetStores(value, oldValue) {
-        if (value) {
-            Object.entries(value).forEach(([key, storeValue]) => {
-                value[key] = ClassSystemUtil.beforeSetInstance(storeValue);
-            });
-        }
+        value && Object.entries(value).forEach(([key, storeValue]) => {
+            value[key] = ClassSystemUtil.beforeSetInstance(storeValue);
+        });
 
         return value;
     }
@@ -257,7 +255,7 @@ class Component extends Base {
                 if (!root[_key]) {
                     Object.defineProperty(root, _key, {
                         enumerable: false,
-                        value     : value,
+                        value,
                         writable  : true
                     });
                 } else {
@@ -542,10 +540,10 @@ class Component extends Base {
         }
 
         me.fire('dataPropertyChange', {
-            key     : key,
-            id      : me.id,
-            oldValue: oldValue,
-            value   : value
+            key,
+            id: me.id,
+            oldValue,
+            value
         });
     }
 
@@ -625,4 +623,4 @@ class Component extends Base {
 
 Neo.applyClassConfig(Component);
 
-export {Component as default};
+export default Component;

package/src/plugin/Base.mjs

@@ -50,4 +50,4 @@ class Base extends CoreBase {
 
 Neo.applyClassConfig(Base);
 
-export {Base as default};
+export default Base;

package/src/plugin/Resizable.mjs

@@ -587,4 +587,4 @@ class Resizable extends Base {
 
 Neo.applyClassConfig(Resizable);
 
-export {Resizable as default};
+export default Resizable;

package/src/selection/CircleModel.mjs

@@ -103,4 +103,4 @@ class CircleModel extends Model {
 
 Neo.applyClassConfig(CircleModel);
 
-export {CircleModel as default};
+export default CircleModel;

package/src/selection/DateSelectorModel.mjs

@@ -174,4 +174,4 @@ class DateSelectorModel extends Model {
 
 Neo.applyClassConfig(DateSelectorModel);
 
-export {DateSelectorModel as default};
+export default DateSelectorModel;

package/src/selection/GalleryModel.mjs

@@ -294,4 +294,4 @@ class GalleryModel extends Model {
 
 Neo.applyClassConfig(GalleryModel);
 
-export {GalleryModel as default};
+export default GalleryModel;

package/src/selection/HelixModel.mjs

@@ -289,4 +289,4 @@ class HelixModel extends Model {
 
 Neo.applyClassConfig(HelixModel);
 
-export {HelixModel as default};
+export default HelixModel;

package/src/selection/ListModel.mjs

@@ -170,4 +170,4 @@ class ListModel extends Model {
 
 Neo.applyClassConfig(ListModel);
 
-export {ListModel as default};
+export default ListModel;

package/src/selection/Model.mjs

@@ -269,4 +269,4 @@ class Model extends Base {
 
 Neo.applyClassConfig(Model);
 
-export {Model as default};
+export default Model;

package/src/selection/menu/ListModel.mjs

@@ -46,4 +46,4 @@ class ListModel extends BaseListModel {
 
 Neo.applyClassConfig(ListModel);
 
-export {ListModel as default};
+export default ListModel;

package/src/selection/table/CellColumnModel.mjs

@@ -119,4 +119,4 @@ class CellColumnModel extends CellModel {
 
 Neo.applyClassConfig(CellColumnModel);
 
-export {CellColumnModel as default};
+export default CellColumnModel;

package/src/selection/table/CellColumnRowModel.mjs

@@ -119,4 +119,4 @@ class CellColumnRowModel extends CellRowModel {
 
 Neo.applyClassConfig(CellColumnRowModel);
 
-export {CellColumnRowModel as default};
+export default CellColumnRowModel;

package/src/selection/table/CellModel.mjs

@@ -181,4 +181,4 @@ class CellModel extends Model {
 
 Neo.applyClassConfig(CellModel);
 
-export {CellModel as default};
+export default CellModel;

package/src/selection/table/CellRowModel.mjs

@@ -161,4 +161,4 @@ class CellRowModel extends CellModel {
 
 Neo.applyClassConfig(CellRowModel);
 
-export {CellRowModel as default};
+export default CellRowModel;

package/src/selection/table/ColumnModel.mjs

@@ -182,4 +182,4 @@ class ColumnModel extends Model {
 
 Neo.applyClassConfig(ColumnModel);
 
-export {ColumnModel as default};
+export default ColumnModel;

package/src/selection/table/RowModel.mjs

@@ -191,4 +191,4 @@ class RowModel extends Model {
 
 Neo.applyClassConfig(RowModel);
 
-export {RowModel as default};
+export default RowModel;

package/src/tab/Container.mjs

@@ -615,4 +615,4 @@ class Container extends BaseContainer {
 
 Neo.applyClassConfig(Container);
 
-export {Container as default};
+export default Container;

package/src/tab/Strip.mjs

@@ -174,4 +174,4 @@ class Strip extends Component {
 
 Neo.applyClassConfig(Strip);
 
-export {Strip as default};
+export default Strip;

package/src/tab/header/Button.mjs

@@ -60,4 +60,4 @@ class Button extends BaseButton {
 
 Neo.applyClassConfig(Button);
 
-export {Button as default};
+export default Button;

package/src/tab/header/Toolbar.mjs

@@ -141,4 +141,4 @@ class Toolbar extends BaseToolbar {
 
 Neo.applyClassConfig(Toolbar);
 
-export {Toolbar as default};
+export default Toolbar;

package/src/table/Container.mjs

@@ -479,4 +479,4 @@ class Container extends BaseContainer {
 
 Neo.applyClassConfig(Container);
 
-export {Container as default};
+export default Container;

package/src/table/View.mjs

@@ -263,4 +263,4 @@ class View extends Component {
 
 Neo.applyClassConfig(View);
 
-export {View as default};
+export default View;

package/src/table/header/Button.mjs

@@ -454,4 +454,4 @@ class Button extends BaseButton {
 
 Neo.applyClassConfig(Button);
 
-export {Button as default};
+export default Button;

package/src/table/header/Toolbar.mjs

@@ -157,4 +157,4 @@ class Toolbar extends BaseToolbar {
 
 Neo.applyClassConfig(Toolbar);
 
-export {Toolbar as default};
+export default Toolbar;

package/src/tooltip/Base.mjs

@@ -263,4 +263,4 @@ class Base extends Container {
 
 Neo.applyClassConfig(Base);
 
-export {Base as default};
+export default Base;

package/src/tree/List.mjs

@@ -414,4 +414,4 @@ class Tree extends Base {
 
 Neo.applyClassConfig(Tree);
 
-export {Tree as default};
+export default Tree;

package/src/util/KeyNavigation.mjs

@@ -78,9 +78,7 @@ class KeyNavigation extends Base {
                 scope = Neo.get(key.scope);
 
                 if (key.key.toUpperCase() === upperCaseKey) {
-                    if (scope[key.fn]) {
-                        scope[key.fn].apply(scope, [data]);
-                    }
+                    scope[key.fn]?.apply(scope, [data]);
                 }
             });
         }
@@ -188,9 +186,7 @@ class KeyNavigation extends Base {
      * @param {Array} items
      */
     removeKeys(items) {
-        if (Array.isArray(items)) {
-            items.forEach(item => this.removeKey(item));
-        }
+        Array.isArray(items) && items.forEach(item => this.removeKey(item));
     }
 
     /**

package/src/vdom/VNode.mjs

@@ -55,4 +55,4 @@ class VNode {
 const ns = Neo.ns('Neo.vdom', true);
 ns['VNode'] = VNode;
 
-export {VNode as default};
+export default VNode;

package/src/worker/App.mjs

@@ -262,24 +262,15 @@ class App extends Base {
 
     /**
      * Only needed for SharedWorkers
-     * @param {String} name
+     * @param {String} appName
      */
-    registerApp(name) {
-        let me = this;
-
-        me.ports.forEach(port => {
-            if (!port.appName) {
-                port.appName = name;
+    registerApp(appName) {
+        // register the name as fast as possible
+        this.onRegisterApp({ appName });
 
-                me.onConnect({
-                    appName: name
-                });
-
-                me.sendMessage('main', {
-                    action :'registerAppName',
-                    appName: name
-                });
-            }
+        this.sendMessage('main', {
+            action:'registerAppName',
+            appName
         });
     }
 

package/src/worker/Base.mjs

@@ -71,8 +71,8 @@ class Base extends CoreBase {
             gt.onmessage = me.onMessage.bind(me);
         }
 
-        Neo.workerId      = me.workerId;
         Neo.currentWorker = me;
+        Neo.workerId      = me.workerId;
     }
 
     /**
@@ -204,6 +204,25 @@ class Base extends CoreBase {
         });
     }
 
+    /**
+     * Only relevant for SharedWorkers
+     * @param {Object} msg
+     * @param {String} msg.appName
+     */
+    onRegisterApp(msg) {
+        let me      = this,
+            appName = msg.appName,
+            port;
+
+        for (port of me.ports) {
+            if (!port.appName) {
+                port.appName = appName;
+                me.onConnect({ appName });
+                break;
+            }
+        }
+    }
+
     /**
      * @param {Object} msg
      */
@@ -275,4 +294,4 @@ class Base extends CoreBase {
 
 Neo.applyClassConfig(Base);
 
-export {Base as default};
+export default Base;

package/src/worker/Manager.mjs

@@ -119,7 +119,7 @@ class Manager extends Base {
 
     /**
      * Sends a message to each worker defined inside the this.workers config.
-     * @param {String} msg
+     * @param {Object} msg
      */
     broadcast(msg) {
         Object.keys(this.workers).forEach(name => {
@@ -261,10 +261,9 @@ class Manager extends Base {
      * @param {Object} e
      */
     onWorkerMessage(e) {
-        let me           = this,
-            data         = e.data,
-            delayPromise = false,
-            transfer     = null,
+        let me       = this,
+            data     = e.data,
+            transfer = null,
             promise;
 
         const {action, destination: dest, replyId} = data;
@@ -273,33 +272,26 @@ class Manager extends Base {
 
         me.fire('message:'+action, data);
 
-        if (promise = action === 'reply' && me.promises[replyId]) {
-            if (data.destination === 'main') {
-                data = data.data;
-            }
+        if (action === 'reply') {
+            promise = me.promises[replyId];
 
-            if (data.reject) {
-                promise.reject(data);
-            } else {
+            if (!promise) {
                 if (data.data) {
-                    if (data.data.autoMount) {
-                        me.fire('automount', data);
-                        delayPromise = true;
-                    }
-                    if (data.data.updateVdom) {
-                        me.fire('updateVdom', data);
-                        delayPromise = true;
-                    }
+                    data.data.autoMount  && me.fire('automount',  data);
+                    data.data.updateVdom && me.fire('updateVdom', data);
+
+                    // we want to delay the message until the rendering queue has processed it
+                    // see: https://github.com/neomjs/neo/issues/2864
+                    me.promiseForwardMessage(data).then(msgData => {
+                        me.sendMessage(msgData.destination, msgData);
+                    });
                 }
-
-                if (!delayPromise) {
-                    promise.resolve(data);
-                } else {
-                    me.promises[replyId].data = data;
+            } else {
+                if (data.destination === 'main') {
+                    data = data.data;
                 }
-            }
 
-            if (!delayPromise) {
+                promise[data.reject ? 'reject' : 'resolve'](data);
                 delete me.promises[replyId];
             }
         }
@@ -324,6 +316,11 @@ class Manager extends Base {
         // only needed for SharedWorkers
         else if (dest === 'main' && action === 'registerAppName') {
             me.appNames.push(data.appName);
+
+            me.broadcast({
+                action : 'registerApp',
+                appName: data.appName
+            });
         }
 
         else if (dest === 'main' && action === 'remoteMethod') {
@@ -332,7 +329,18 @@ class Manager extends Base {
     }
 
     /**
-     * @param {String} dest app, data or vdom
+     * @param {Object} data
+     * @param {String} data.replyId
+     * @returns {Promise<any>}
+     */
+    promiseForwardMessage(data) {
+        return new Promise((resolve, reject) => {
+            this.promises[data.replyId] = { data, reject, resolve };
+        });
+    }
+
+    /**
+     * @param {String} dest app, canvas, data or vdom
      * @param {Object} opts configs for Neo.worker.Message
      * @param {Array} [transfer] An optional array of Transferable objects to transfer ownership of.
      * If the ownership of an object is transferred, it becomes unusable (neutered) in the context it was sent from
@@ -346,10 +354,7 @@ class Manager extends Base {
             let message = me.sendMessage(dest, opts, transfer),
                 msgId   = message.id;
 
-            me.promises[msgId] = {
-                reject,
-                resolve
-            };
+            me.promises[msgId] = { reject, resolve };
         });
     }
 
@@ -368,7 +373,7 @@ class Manager extends Base {
     }
 
     /**
-     * @param {String} dest app, data or vdom
+     * @param {String} dest app, canvas, data or vdom
      * @param {Object} opts configs for Neo.worker.Message
      * @param {Array} [transfer] An optional array of Transferable objects to transfer ownership of.
      * If the ownership of an object is transferred, it becomes unusable (neutered) in the context it was sent from

package/src/worker/Message.mjs

@@ -37,4 +37,4 @@ class Message {
 const ns = Neo.ns('Neo.worker', true);
 ns['Message'] = Message;
 
-export {Message as default};
+export default Message;

package/src/worker/VDom.mjs

@@ -28,6 +28,19 @@ class VDom extends Base {
          */
         workerId: 'vdom'
     }}
+
+    /**
+     *
+     */
+    afterConnect() {
+        let me      = this,
+            channel = new MessageChannel(),
+            port    = channel.port2;
+
+        channel.port1.onmessage = me.onMessage.bind(me);
+
+        me.sendMessage('app', {action: 'registerPort', transfer: port}, [port]);
+    }
 }
 
 Neo.applyClassConfig(VDom);

package/src/worker/mixin/RemoteMethodAccess.mjs

@@ -155,4 +155,4 @@ class RemoteMethodAccess extends Base {
 
 Neo.applyClassConfig(RemoteMethodAccess);
 
-export {RemoteMethodAccess as default};
+export default RemoteMethodAccess;

package/test/siesta/tests/CollectionBase.mjs

@@ -160,11 +160,11 @@ StartTest(t => {
             {country: 'Croatia',   firstname: 'Grgur',   githubId: 'grgur',         lastname: 'Grisogono'},
             {country: 'Slovakia',  firstname: 'Jozef',   githubId: 'jsakalos',      lastname: 'Sakalos'},
             {country: 'Argentina', firstname: 'Max',     githubId: 'elmasse',       lastname: 'Fierro'},
+            {country: 'France',    firstname: 'Nigel',   githubId: 'NigeWhite',     lastname: 'White'},
             {country: 'Germany',   firstname: 'Nils',    githubId: 'mrsunshine',    lastname: 'Dehl'},
             {country: 'USA',       firstname: 'Rich',    githubId: 'rwaters',       lastname: 'Waters'},
             {country: 'Germany',   firstname: 'Tobias',  githubId: 'tobiu2',        lastname: 'Uhlig2'},
-            {country: 'Germany',   firstname: 'Tobias',  githubId: 'tobiu',         lastname: 'Uhlig'},
-            {country: 'France',    firstname: 'Nigel',   githubId: 'NigeWhite',     lastname: 'White'}
+            {country: 'Germany',   firstname: 'Tobias',  githubId: 'tobiu',         lastname: 'Uhlig'}
         ], 'collection2.getRange()');
 
         t.notOk(collection2.isFiltered(), 'collection2.isFiltered()');
@@ -186,11 +186,11 @@ StartTest(t => {
             {country: 'Croatia',   firstname: 'Grgur',   githubId: 'grgur',         lastname: 'Grisogono'},
             {country: 'Slovakia',  firstname: 'Jozef',   githubId: 'jsakalos',      lastname: 'Sakalos'},
             {country: 'Argentina', firstname: 'Max',     githubId: 'elmasse',       lastname: 'Fierro'},
+            {country: 'France',    firstname: 'Nigel',   githubId: 'NigeWhite',     lastname: 'White'},
             {country: 'Germany',   firstname: 'Nils',    githubId: 'mrsunshine',    lastname: 'Dehl'},
             {country: 'USA',       firstname: 'Rich',    githubId: 'rwaters',       lastname: 'Waters'},
             {country: 'Germany',   firstname: 'Tobias',  githubId: 'tobiu2',        lastname: 'Uhlig2'},
             {country: 'Germany',   firstname: 'Tobias',  githubId: 'tobiu',         lastname: 'Uhlig'},
-            {country: 'France',    firstname: 'Nigel',   githubId: 'NigeWhite',     lastname: 'White'}
         ], 'collection2.getRange()');
 
         collection2.clearSorters(true);
@@ -251,4 +251,4 @@ StartTest(t => {
         t.isStrict(collection3.getCount(), 5, 'collection3 count is 5');
         t.isStrict(collection3.allItems.getCount(), 9, 'collection3 allItems count is 9');
     });
-});
+});