neo.mjs 10.0.0-alpha.5 → 10.0.0-beta.2

package/src/main/addon/MonacoEditor.mjs

@@ -16,6 +16,20 @@ class MonacoEditor extends Base {
          * @protected
          */
         className: 'Neo.main.addon.MonacoEditor',
+        /**
+         * List methods which must get cached until the addon reaches its `isReady` state
+         * @member {String[]} interceptRemotes
+         */
+        interceptRemotes: [
+            'createInstance',
+            'destroyInstance',
+            'getValue',
+            'layoutEditor',
+            'setLanguage',
+            'setTheme',
+            'setValue',
+            'updateOptions'
+        ],
         /**
          * @member {String} libraryBasePath='../../node_modules/monaco-editor/min/vs'
          */
@@ -53,23 +67,18 @@ class MonacoEditor extends Base {
     createInstance(data) {
         let me   = this,
             {id} = data,
-            editor, node;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'createInstance', data})
-        } else {
-            delete data.appName;
-            delete data.id;
+            node = DomAccess.getElement(id),
+            editor;
 
-            node = DomAccess.getElement(id);
+        delete data.appName;
+        delete data.id;
 
-            if (node) {
-                editor = me.map[id] = monaco.editor.create(node, data);
+        if (node) {
+            editor = me.map[id] = monaco.editor.create(node, data);
 
-                editor.getModel().onDidChangeContent(me.onContentChange.bind(me, id))
-            } else if (Neo.config.environment === 'development') {
-                console.warn(`addon.MonacoEditor: node ${id} not found`)
-            }
+            editor.getModel().onDidChangeContent(me.onContentChange.bind(me, id))
+        } else if (Neo.config.environment === 'development') {
+            console.warn(`addon.MonacoEditor: node ${id} not found`)
         }
     }
 
@@ -78,14 +87,8 @@ class MonacoEditor extends Base {
      * @param {String} data.id
      */
     destroyInstance(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'destroyInstance', data})
-        } else {
-            // todo: destroy the editor instance if possible
-            delete this.map[data.id]
-        }
+        // todo: destroy the editor instance if possible
+        delete this.map[data.id]
     }
 
     /**
@@ -94,13 +97,7 @@ class MonacoEditor extends Base {
      * @returns {Object}
      */
     getValue(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'getValue', data})
-        } else {
-            return me.map[data.id].getModel().getValue()
-        }
+        return this.map[data.id].getModel().getValue()
     }
 
     /**
@@ -109,7 +106,7 @@ class MonacoEditor extends Base {
      * @param {String} data.id
      */
     layoutEditor(data) {
-        this.isReady && this.map[data.id].layout()
+        this.map[data.id].layout()
     }
 
     /**
@@ -119,8 +116,6 @@ class MonacoEditor extends Base {
         let me   = this,
             path = me.libraryBasePath;
 
-        me.isLoading = true;
-
         window.require = {paths: {vs: path}};
 
         await DomAccess.loadScript(path + '/loader.js');
@@ -129,10 +124,7 @@ class MonacoEditor extends Base {
             DomAccess.loadStylesheet(path + '/editor/editor.main.css', {name: 'vs/editor/editor.main'}),
             DomAccess.loadScript(path + '/editor/editor.main.nls.js'),
             DomAccess.loadScript(path + '/editor/editor.main.js')
-        ]);
-
-        me.isLoading = false;
-        me.isReady   = true
+        ])
     }
 
     /**
@@ -159,13 +151,7 @@ class MonacoEditor extends Base {
      * @param {String} data.value
      */
     setLanguage(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'setLanguage', data})
-        } else {
-            me.map[data.id].getModel().setLanguage(data.value)
-        }
+        this.map[data.id].getModel().setLanguage(data.value)
     }
 
     /**
@@ -174,13 +160,7 @@ class MonacoEditor extends Base {
      * @param {String} data.value
      */
     setTheme(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'setTheme', data})
-        } else {
-            me.map[data.id]._themeService.setTheme(data.value)
-        }
+        this.map[data.id]._themeService.setTheme(data.value)
     }
 
     /**
@@ -189,13 +169,7 @@ class MonacoEditor extends Base {
      * @param {String} data.value
      */
     setValue(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'setValue', data})
-        } else {
-            me.map[data.id].getModel().setValue(data.value)
-        }
+        this.map[data.id].getModel().setValue(data.value)
     }
 
     /**
@@ -204,13 +178,7 @@ class MonacoEditor extends Base {
      * @param {Object} data.options
      */
     updateOptions(data) {
-        let me = this;
-
-        if (!me.isReady) {
-            return me.cacheMethodCall({fn: 'updateOptions', data})
-        } else {
-            me.map[data.id].updateOptions(data.options)
-        }
+        this.map[data.id].updateOptions(data.options)
     }
 }
 

package/src/manager/ClassHierarchy.mjs

@@ -0,0 +1,114 @@
+import BaseManager from './Base.mjs';
+
+/**
+ * @class Neo.manager.ClassHierarchy
+ * @extends Neo.manager.Base
+ * @singleton
+ *
+ * This manager maintains a registry of all classes defined within the Neo.mjs framework's current realm (main or worker),
+ * including their inheritance relationships and key metadata.
+ * Each registered item (value in the manager's store) has the following structure:
+ * @typedef {Object} ClassHierarchyInfo
+ * @property {String} className - The full Neo.mjs class name (e.g., 'Neo.component.Base').
+ * @property {Function|Object} module - The direct reference to the class constructor function itself (for non-singletons)
+ * or the instantiated singleton object (for singletons).
+ * @property {String|null} ntype - The ntype of the class if defined (e.g., 'button', 'container'), otherwise `null`.
+ * @property {String|null} parentClassName - The full class name of its direct parent class,
+ * or `null` if it's a top-level class (e.g., 'Neo.core.Base').
+ */
+class ClassHierarchy extends BaseManager {
+    static config = {
+        /**
+         * @member {String} className='Neo.manager.ClassHierarchy'
+         * @protected
+         */
+        className: 'Neo.manager.ClassHierarchy',
+        /**
+         * @member {String} keyProperty='className'
+         * @protected
+         */
+        keyProperty: 'className',
+        /**
+         * @member {Boolean} singleton=true
+         * @protected
+         */
+        singleton: true
+    }
+
+    /**
+     * Memoizes the return values of isA() calls
+     * @member {Map} isAQueryMap=new Map()
+     * @protected
+     */
+    isAQueryMap = new Map()
+
+    /**
+     * @param {Object} config
+     */
+    construct(config) {
+        super.construct(config);
+        this.consumeTempMap()
+    }
+
+    /**
+     * Register all classes that got applied to the Neo namespace before this instance got created
+     * @protected
+     */
+    consumeTempMap() {
+        if (Neo.classHierarchyMap) {
+            this.add(Object.values(Neo.classHierarchyMap));
+            delete Neo.classHierarchyMap
+        }
+    }
+
+    /**
+     * Both params represent classNames.
+     *
+     * Example use cases:
+     * - isA('Neo.button.Menu',    'Neo.button.Base')    => true
+     * - isA('Neo.button.Base',    'Neo.button.Menu')    => false
+     * - isA('Neo.button.Base',    'Neo.component.Base') => true
+     * - isA('Neo.component.Base', 'Neo.core.Base')      => true
+     * @param {String} descendant
+     * @param {String} ancestor
+     * @returns {Boolean}
+     */
+    isA(descendant, ancestor) {
+        if (descendant === ancestor) {
+            return true
+        }
+
+        let parent        = descendant,
+            {isAQueryMap} = this,
+            queryName     = `${descendant},${ancestor}`,
+            returnValue   = false;
+
+        if (isAQueryMap.has(queryName)) {
+            return isAQueryMap.get(queryName)
+        }
+
+        while (parent = this.get(parent)?.parentClassName) {
+            if (parent === ancestor) {
+                returnValue = true;
+                break
+            }
+
+            // Assumption: component.Base directly extends core.Base
+            if (parent === 'Neo.component.Base' && ancestor !== 'Neo.core.Base') {
+                returnValue = false;
+                break
+            }
+
+            if (parent === 'Neo.core.Base') {
+                returnValue = false;
+                break
+            }
+        }
+
+        isAQueryMap.set(queryName, returnValue);
+
+        return returnValue
+    }
+}
+
+export default Neo.setupClass(ClassHierarchy);

package/src/menu/List.mjs

@@ -182,7 +182,7 @@ class List extends BaseList {
         let me        = this,
             {iconCls} = record,
             id        = record[me.store.keyProperty],
-            vdomCn    = [{tag: 'span', cls: ['neo-content'], html: record[me.displayField]}];
+            vdomCn    = [{tag: 'span', cls: ['neo-content'], text: record[me.displayField]}];
 
         if (iconCls && iconCls !== '') {
             vdomCn.unshift({tag: 'i', cls: ['neo-menu-icon', 'neo-icon', iconCls], id: me.getIconId(id)})

package/src/plugin/Popover.mjs

@@ -18,10 +18,10 @@ import Plugin    from './Base.mjs';
  *             ntype  : 'panel',
  *             headers: [{
  *                 dock: 'top',
- *                 html: 'HEADER'
+ *                 text: 'HEADER'
  *             }],
  *             items  : [{
- *                 html: 'This is a comment about the button'
+ *                 text: 'This is a comment about the button'
  *             }]
  *         }]
  *     }]

package/src/sitemap/Component.mjs

@@ -130,7 +130,7 @@ class Component extends Base {
                 tag : 'a',
                 cls : ['neo-action', `neo-level-${record.level}`],
                 id  : me.getItemId(record.id),
-                html: record.name
+                text: record.name
             };
 
             if (action && action !== '') {

package/src/table/{View.mjs → Body.mjs}

@@ -5,25 +5,25 @@ import RowModel        from '../selection/table/RowModel.mjs';
 import VDomUtil        from '../util/VDom.mjs';
 
 /**
- * @class Neo.table.View
+ * @class Neo.table.Body
  * @extends Neo.component.Base
  */
-class View extends Component {
+class TableBody extends Component {
     static config = {
         /**
-         * @member {String} className='Neo.table.View'
+         * @member {String} className='Neo.table.Body'
          * @protected
          */
-        className: 'Neo.table.View',
+        className: 'Neo.table.Body',
         /**
-         * @member {String} ntype='table-view'
+         * @member {String} ntype='table-body'
          * @protected
          */
-        ntype: 'table-view',
+        ntype: 'table-body',
         /**
-         * @member {String[]} baseCls=['neo-table-view']
+         * @member {String[]} baseCls=['neo-table-body']
          */
-        baseCls: ['neo-table-view'],
+        baseCls: ['neo-table-body'],
         /**
          * Define which model field contains the value of colspan definitions
          * @member {String} colspanField='colspan'
@@ -132,18 +132,17 @@ class View extends Component {
     }
 
     /**
-     * @param {Object} data
-     * @param {String} [data.cellId]
-     * @param {Object} data.column
-     * @param {Number} data.columnIndex
-     * @param {Object} data.record
-     * @param {Number} data.rowIndex
+     * @param {Object}              data
+     * @param {String}              [data.cellId]
+     * @param {Object}              data.column
+     * @param {Number}              data.columnIndex
+     * @param {Object}              data.record
+     * @param {Number}              data.rowIndex
      * @param {Neo.table.Container} data.tableContainer
      * @returns {Object}
      */
-    applyRendererOutput(data) {
-        let {cellId, column, columnIndex, record, rowIndex, tableContainer} = data,
-            me          = this,
+    applyRendererOutput({cellId, column, columnIndex, record, rowIndex, tableContainer}) {
+        let me          = this,
             cellCls     = ['neo-table-cell'],
             colspan     = record[me.colspanField],
             {dataField} = column,
@@ -178,7 +177,7 @@ class View extends Component {
 
         switch (Neo.typeOf(rendererOutput)) {
             case 'Object': {
-                if (rendererOutput.html) {
+                if (rendererOutput.html || rendererOutput.text) {
                     rendererOutput.cls && cellCls.push(...rendererOutput.cls);
                 } else {
                     rendererOutput = [rendererOutput];
@@ -231,7 +230,11 @@ class View extends Component {
         }
 
         if (Neo.typeOf(rendererOutput) === 'Object') {
-            cellConfig.html = rendererOutput.html  || ''
+            if (Object.hasOwn(rendererOutput, 'html')) {
+                cellConfig.html = rendererOutput.html  || ''
+            } else {
+                cellConfig.text = rendererOutput.text  || ''
+            }
         } else {
             cellConfig.cn = rendererOutput
         }
@@ -382,7 +385,7 @@ class View extends Component {
             dataField = me.getCellDataField(id),
             record    = me.getRecord(id);
 
-        me.parent.fire(eventName, {data, dataField, record, view: me})
+        me.parent.fire(eventName, {body: me, data, dataField, record})
     }
 
     /**
@@ -394,7 +397,7 @@ class View extends Component {
             id     = data.currentTarget,
             record = me.getRecord(id);
 
-        me.parent.fire(eventName, {data, record, view: me})
+        me.parent.fire(eventName, {body: me, data, record})
     }
 
     /**
@@ -619,4 +622,4 @@ class View extends Component {
     }
 }
 
-export default Neo.setupClass(View);
+export default Neo.setupClass(TableBody);

package/src/table/Container.mjs

@@ -3,7 +3,7 @@ import ClassSystemUtil from '../util/ClassSystem.mjs';
 import CssUtil         from '../util/Css.mjs';
 import NeoArray        from '../util/Array.mjs';
 import Store           from '../data/Store.mjs';
-import View            from './View.mjs';
+import TableBody       from './Body.mjs';
 import * as header     from './header/_export.mjs';
 
 /**
@@ -26,6 +26,16 @@ class Container extends BaseContainer {
          * @member {String[]} baseCls=['neo-table-container']
          */
         baseCls: ['neo-table-container'],
+        /**
+         * Configs for Neo.table.Body
+         * @member {Object|null} [bodyConfig=null]
+         */
+        bodyConfig: null,
+        /**
+         * @member {String|null} bodyId_=null
+         * @protected
+         */
+        bodyId_: null,
         /**
          * true uses table.plugin.CellEditing
          * @member {Boolean} cellEditing_=false
@@ -75,16 +85,6 @@ class Container extends BaseContainer {
          * @member {Boolean} useCustomScrollbars_=true
          */
         useCustomScrollbars_: true,
-        /**
-         * Configs for Neo.table.View
-         * @member {Object|null} [viewConfig=null]
-         */
-        viewConfig: null,
-        /**
-         * @member {String|null} viewId_=null
-         * @protected
-         */
-        viewId_: null,
         /**
          * @member {Array|null} items=null
          * @protected
@@ -100,19 +100,19 @@ class Container extends BaseContainer {
     }
 
     /**
-     * Convenience method to access the Neo.table.header.Toolbar
-     * @returns {Neo.table.header.Toolbar|null}
+     * Convenience method to access the Neo.table.Body
+     * @returns {Neo.table.Body|null}
      */
-    get headerToolbar() {
-        return Neo.getComponent(this.headerToolbarId) || Neo.get(this.headerToolbarId)
+    get body() {
+        return Neo.getComponent(this.bodyId) || Neo.get(this.bodyId)
     }
 
     /**
-     * Convenience method to access the Neo.table.View
-     * @returns {Neo.table.View|null}
+     * Convenience method to access the Neo.table.header.Toolbar
+     * @returns {Neo.table.header.Toolbar|null}
      */
-    get view() {
-        return Neo.getComponent(this.viewId) || Neo.get(this.viewId)
+    get headerToolbar() {
+        return Neo.getComponent(this.headerToolbarId) || Neo.get(this.headerToolbarId)
     }
 
     /**
@@ -123,8 +123,8 @@ class Container extends BaseContainer {
 
         let me = this;
 
+        me.bodyId          = Neo.getId('table-body');
         me.headerToolbarId = Neo.getId('table-header-toolbar');
-        me.viewId          = Neo.getId('table-view');
 
         me.items = [{
             module           : header.Toolbar,
@@ -133,11 +133,11 @@ class Container extends BaseContainer {
             sortable         : me.sortable,
             ...me.headerToolbarConfig
         }, {
-            module     : View,
+            module     : TableBody,
             containerId: me.id,
-            id         : me.viewId,
+            id         : me.bodyId,
             store      : me.store,
-            ...me.viewConfig
+            ...me.bodyConfig
         }];
 
         me.vdom.id = me.getWrapperId();
@@ -184,7 +184,7 @@ class Container extends BaseContainer {
                 headerToolbar.createItems()
             }
 
-            me.view?.createViewData()
+            me.body?.createViewData()
         }
     }
 
@@ -229,9 +229,9 @@ class Container extends BaseContainer {
         value   ?.on(listeners);
         oldValue?.un(listeners);
 
-        // in case we dynamically change the store, the view needs to get the new reference
-        if (me.view) {
-            me.view.store = value
+        // in case we dynamically change the store, the body needs to get the new reference
+        if (me.body) {
+            me.body.store = value
         }
     }
 
@@ -270,6 +270,16 @@ class Container extends BaseContainer {
         me.scrollbarsCssApplied = true
     }
 
+    /**
+     * Triggered before the bodyId config gets changed.
+     * @param {String} value
+     * @param {String} oldValue
+     * @protected
+     */
+    beforeSetBodyId(value, oldValue) {
+        return value || oldValue
+    }
+
     /**
      * Triggered before the columns config gets changed.
      * @param {Object[]} value
@@ -308,16 +318,6 @@ class Container extends BaseContainer {
         return value
     }
 
-    /**
-     * Triggered before the viewId config gets changed.
-     * @param {String} value
-     * @param {String} oldValue
-     * @protected
-     */
-    beforeSetViewId(value, oldValue) {
-        return value || oldValue
-    }
-
     /**
      * In case you want to update multiple existing records in parallel,
      * using this method is faster than updating each record one by one.
@@ -325,19 +325,19 @@ class Container extends BaseContainer {
      * @param {Object[]} records
      */
     bulkUpdateRecords(records) {
-        let {store, view} = this,
+        let {body, store} = this,
             {keyProperty} = store;
 
-        if (view) {
-            view.silentVdomUpdate = true;
+        if (body) {
+            body.silentVdomUpdate = true;
 
             records.forEach(item => {
                 store.get(item[keyProperty])?.set(item)
             });
 
-            view.silentVdomUpdate = false;
+            body.silentVdomUpdate = false;
 
-            view.update()
+            body.update()
         }
     }
 
@@ -426,7 +426,7 @@ class Container extends BaseContainer {
 
         me.store.sort(opts);
         me.removeSortingCss(opts.property);
-        opts.direction && me.view.onStoreLoad()
+        opts.direction && me.body.onStoreLoad()
     }
 
     /**

package/src/table/_export.mjs

@@ -1,5 +1,5 @@
 import * as header from './header/_export.mjs';
+import Body        from './Body.mjs';
 import Container   from './Container.mjs';
-import View        from './View.mjs';
 
-export {header, Container, View};
+export {header, Body, Container};