@truenas/ui-components 0.1.73 → 0.1.75

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truenas/ui-components",
-  "version": "0.1.73",
+  "version": "0.1.75",
   "publishConfig": {
     "registry": "https://registry.npmjs.org",
     "access": "public"

package/types/truenas-ui-components.d.ts

@@ -40,20 +40,24 @@ declare function kebabTestSegment(part: string | number): string;
  * Normalizes the base — a single token or an array of segments — into a flat
  * segment array and appends the suffixes, producing a value that is itself a
  * valid {@link TnTestIdValue}. This is the canonical "derive a per-child id from
- * a parent base" operation, e.g. a dialog's close button (`[base, 'close']`) or
- * a select's per-option id (`[base, option.label]`). Centralizing it keeps the
- * `string | array` flattening contract in one place rather than re-implemented
- * at each call site.
+ * a parent base" operation, e.g. a select's per-option id
+ * (`[base, option.label]`) or a menu item's id (`[base, item.id]`). Centralizing
+ * it keeps the `string | array` flattening contract in one place rather than
+ * re-implemented at each call site.
+ *
+ * Note this is *base-first*: the suffix trails the base. Fixed chrome whose role
+ * should lead (e.g. `tn-dialog-shell`'s `button-close-<base>`) prepends the role
+ * manually instead of calling this.
  *
  * Falsy segments are preserved here (not filtered) so `composeTestId` can apply
- * its own drop/scoping rules — `scopeTestId(undefined, 'close')` yields
- * `[undefined, 'close']`, which `composeTestId('button', …)` renders as the
- * unscoped `button-close`.
+ * its own drop/scoping rules — `scopeTestId(undefined, 'edit')` yields
+ * `[undefined, 'edit']`, which `composeTestId('menu-item', …)` renders as the
+ * unscoped `menu-item-edit`.
  *
  * @example
  * scopeTestId('actions', 'edit')        // ['actions', 'edit']
  * scopeTestId(['menu', 'main'], 'edit') // ['menu', 'main', 'edit']
- * scopeTestId(undefined, 'close')       // [undefined, 'close']
+ * scopeTestId(undefined, 'edit')        // [undefined, 'edit']
  */
 declare function scopeTestId(base: TnTestIdValue, ...suffix: (string | number | null | undefined)[]): (string | number | null | undefined)[];
 /**
@@ -6223,13 +6227,28 @@ declare class TnDialogShellComponent implements OnInit {
     /**
      * Optional semantic base that scopes the shell's chrome buttons. The close
      * and fullscreen buttons emit `button-close` / `button-fullscreen` by default,
-     * or `button-<testId>-close` / `-fullscreen` when a base is provided (useful
-     * when more than one dialog can be open).
+     * or `button-close-<testId>` / `button-fullscreen-<testId>` when a base is
+     * provided (useful when more than one dialog can be open).
      */
     testId: _angular_core.InputSignal<TnTestIdValue>;
-    /** Scoped test-id segments for the close button (`button-[<base>-]close`). */
+    /**
+     * The `testId` base normalized to a flat segment array. Nothing is dropped
+     * here — `composeTestId` (via the `[tnTestId]` directive) filters falsy
+     * segments, so an unset base collapses to the bare role (`button-close`).
+     */
+    private readonly baseSegments;
+    /**
+     * Role-first test-id segments for the close button: `button-close[-<base>]`.
+     *
+     * The close and fullscreen buttons are fixed dialog chrome, not content
+     * children, so the role leads rather than the base (content children are
+     * base-first via `scopeTestId`). This keeps every dialog's close button under
+     * a shared `button-close-*` prefix — it matches webui's established
+     * close-button ids and lets automation target "all close buttons" with one
+     * selector.
+     */
     protected closeTestId: _angular_core.Signal<(string | number | null | undefined)[]>;
-    /** Scoped test-id segments for the fullscreen button (`button-[<base>-]fullscreen`). */
+    /** Role-first test-id segments for the fullscreen button: `button-fullscreen[-<base>]`. */
     protected fullscreenTestId: _angular_core.Signal<(string | number | null | undefined)[]>;
     /** Stable id for the title heading, referenced by the dialog's aria-labelledby. */
     readonly titleId: string;