@uxland/primary-shell 7.41.3 → 7.41.4

package/dist/primary/shell/src/UI/shared-components/primaria-region/primaria-region.d.ts

@@ -60,6 +60,31 @@ export declare class PrimariaRegion extends PrimariaRegion_base {
      * Sets up the region definition before the parent connectedCallback runs.
      */
     connectedCallback(): void;
+    /**
+     * Mixin hook fired after `createRegions` finishes.
+     *
+     * Hydrates the freshly-created region with every view that has already been
+     * registered for this region name via `regionManager.registerViewWithRegion`.
+     * `registerViewWithRegion` only forwards to regions that exist at the moment
+     * of the call, so a plugin that registers once at `initialize` would lose
+     * its view every time the region is destroyed and re-created (e.g. when a
+     * drawer that hosts the region is closed and reopened). Pulling from the
+     * registry here makes injection transparent: any plugin can register once
+     * and any region with that name auto-populates.
+     */
+    regionsCreated(_regions: unknown): void;
+    /**
+     * Called when the component is removed from the DOM.
+     *
+     * The base mixin's `disconnectedCallback` removes the region from the manager
+     * but does NOT clear `this.region`, so on a later reconnect the mixin's
+     * `createRegions` sees `isNil(this.region) === false` and skips re-creating
+     * the region. That breaks the "drawer opens, closes, opens again" flow:
+     * the second open would render the host but never receive the plugin's
+     * view. Clear the reference (and drop the orphan container) so the next
+     * connect rebuilds everything from scratch.
+     */
+    disconnectedCallback(): void;
     /**
      * Called before the component updates.
      * Updates the region definition if the name changes.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxland/primary-shell",
-  "version": "7.41.3",
+  "version": "7.41.4",
   "description": "Primaria Shell",
   "author": "UXLand <dev@uxland.es>",
   "homepage": "https://github.com/uxland/harmonix/tree/app#readme",

package/src/UI/shared-components/primaria-region/primaria-region.ts

@@ -1,6 +1,6 @@
 import { LitElement, html } from "lit";
 import { property } from "lit/decorators.js";
-import { IRegion } from "@uxland/regions";
+import { IRegion, regionManager } from "@uxland/regions";
 import { PrimariaRegionHost } from "../../../api/api";
 import { regionsProperty } from "@uxland/regions/region-decorator";
 
@@ -130,6 +130,47 @@ export class PrimariaRegion extends PrimariaRegionHost(LitElement) {
     }
   }
 
+  /**
+   * Mixin hook fired after `createRegions` finishes.
+   *
+   * Hydrates the freshly-created region with every view that has already been
+   * registered for this region name via `regionManager.registerViewWithRegion`.
+   * `registerViewWithRegion` only forwards to regions that exist at the moment
+   * of the call, so a plugin that registers once at `initialize` would lose
+   * its view every time the region is destroyed and re-created (e.g. when a
+   * drawer that hosts the region is closed and reopened). Pulling from the
+   * registry here makes injection transparent: any plugin can register once
+   * and any region with that name auto-populates.
+   */
+  regionsCreated(_regions: unknown): void {
+    if (!this.region || !this.name) return;
+    const registered = regionManager.getRegisteredViews(this.name) ?? [];
+    for (const { key, view } of registered) {
+      if (this.region.containsView(key)) continue;
+      this.region.addView(key, view).catch((e) => {
+        console.warn(`primaria-region(${this.name}): failed to addView "${key}"`, e);
+      });
+    }
+  }
+
+  /**
+   * Called when the component is removed from the DOM.
+   *
+   * The base mixin's `disconnectedCallback` removes the region from the manager
+   * but does NOT clear `this.region`, so on a later reconnect the mixin's
+   * `createRegions` sees `isNil(this.region) === false` and skips re-creating
+   * the region. That breaks the "drawer opens, closes, opens again" flow:
+   * the second open would render the host but never receive the plugin's
+   * view. Clear the reference (and drop the orphan container) so the next
+   * connect rebuilds everything from scratch.
+   */
+  disconnectedCallback(): void {
+    super.disconnectedCallback();
+    this.region = undefined;
+    const targetId = `${this.name}-container`;
+    this.querySelector(`#${targetId}`)?.remove();
+  }
+
   /**
    * Called before the component updates.
    * Updates the region definition if the name changes.