component.Base, container.Base: afterSetTheme() #6292

Author: tobiu

src/component/Base.mjs

@@ -970,12 +970,16 @@ class Component extends Base {
      */
     afterSetTheme(value, oldValue) {
         if (value || oldValue !== undefined) {
-            let cls = this.cls;
+            let me    = this,
+                {cls} = me;
 
-            NeoArray.remove(cls, oldValue);
-            value && NeoArray.add(cls, value);
+            // We do not need to add a DOM based CSS selector, in case the theme is already inherited
+            if (value !== me.parent?.theme) {
+                NeoArray.remove(cls, oldValue);
+                value && NeoArray.add(cls, value);
 
-            this.cls = cls
+                me.cls = cls
+            }
         }
     }
 

src/container/Base.mjs

@@ -216,6 +216,22 @@ class Container extends Component {
         }
     }
 
+    /**
+     * Triggered after the theme config got changed
+     * @param {String|null} value
+     * @param {String|null} oldValue
+     * @protected
+     */
+    afterSetTheme(value, oldValue) {
+        super.afterSetTheme(value, oldValue);
+
+        value && this.items?.forEach(item => {
+            if (!Neo.isString(item)) {
+                item.theme = value
+            }
+        })
+    }
+
     /**
      * Triggered after the windowId config got changed
      * @param {Number|null} value