@fusion.dev/helix-headless 0.53.1 → 0.53.3

package/dist/tree/root/tree-root.d.ts

@@ -74,14 +74,15 @@ export declare class HelixHeadlessTree extends HelixHeadlessTree_base {
     /**
      * Every item, in the order a reader moving down would meet them.
      *
-     * Selected by tag, never by `'*'` and a filter: this is the array a single
-     * arrow press rebuilds, and `'*'` makes it every descendant element in the
-     * tree, labels and icons and whatever a consumer slotted into a row included.
+     * **By identity rather than by tag**, which is the same reason Splitter
+     * writes out at `findOuter`: a tag list names one tier's element from the
+     * other's package, and misses any third tag, which is exactly what
+     * subclassing an item produces. The item then never enters the array a single
+     * arrow press rebuilds, so it is silently unreachable from the keyboard.
      *
-     * The two tag names rather than `instanceof` on everything: a styled
-     * `helix-tree-item` extends the base, so both names are needed, and the
-     * `instanceof` stays as the type guard because a consumer can write either
-     * tag and neither selector proves what the element is.
+     * `'*'` and a filter rather than a tag: the walk is one native pass and the
+     * `instanceof` is what narrows it, so the labels, icons and whatever a
+     * consumer slotted into a row are gone by the time the array exists.
      */
     private get items();
     /** Items a reader can actually reach: not inside a collapsed branch. */

package/dist/tree/root/tree-root.js

@@ -81,7 +81,7 @@ let HelixHeadlessTree = class HelixHeadlessTree extends MixinCollection(MixinCon
         };
     }
     get items() {
-        return [...this.querySelectorAll('helix-tree-item, helix-headless-tree-item')].filter((node) => node instanceof HelixHeadlessTreeItem);
+        return [...this.querySelectorAll('*')].filter((node) => node instanceof HelixHeadlessTreeItem);
     }
     get visibleItems() {
         return this.items.filter((item) => {

package/dist/video-playlist/menu/video-playlist-menu.d.ts

@@ -61,7 +61,13 @@ export declare class HelixHeadlessVideoPlaylistMenu extends HelixElement {
     private foundPlaylist;
     /** The playlist this menu belongs to, found upward through the slots. */
     private get playlist();
-    /** The playlist's entries, in the order it holds them. */
+    /**
+     * The playlist's entries, in the order it holds them.
+     *
+     * By identity rather than by tag, the same reason the playlist itself reads
+     * them that way: a tag list names one tier's element from the other's package
+     * and misses any third one.
+     */
     private get entries();
     /**
      * Watches the playlist for entries arriving, and re-reads it on every open.

package/dist/video-playlist/menu/video-playlist-menu.js

@@ -9,9 +9,9 @@ import { list } from '@fusion.dev/helix-icons';
 import { nothing } from 'lit';
 import { property } from 'lit/decorators.js';
 import { html } from 'lit/static-html.js';
+import { HelixHeadlessVideoPlaylistItem } from '../item/video-playlist-item.js';
 import { HelixHeadlessVideoPlaylist } from '../root/video-playlist-root.js';
 import '../../menu/index.js';
-const ITEMS = 'helix-video-playlist-item, helix-headless-video-playlist-item';
 let HelixHeadlessVideoPlaylistMenu = class HelixHeadlessVideoPlaylistMenu extends HelixElement {
     constructor() {
         super(...arguments);
@@ -39,7 +39,9 @@ let HelixHeadlessVideoPlaylistMenu = class HelixHeadlessVideoPlaylistMenu extend
     }
     get entries() {
         const playlist = this.playlist;
-        return playlist ? [...playlist.querySelectorAll(ITEMS)] : [];
+        return playlist
+            ? [...playlist.querySelectorAll('*')].filter((one) => one instanceof HelixHeadlessVideoPlaylistItem)
+            : [];
     }
     watch() {
         this.foundPlaylist ??= closestInstance(this, HelixHeadlessVideoPlaylist) ?? undefined;

package/dist/video-playlist/root/video-playlist-root.d.ts

@@ -35,7 +35,7 @@ declare const HelixHeadlessVideoPlaylist_base: (new (...args: any[]) => import("
  * @documentation https://fusion.dev/helix/components/video-playlist
  *
  * @slot - The entries.
- * @slot player - The `helix-video` they play in.
+ * @slot player - The player the entries take turns in.
  *
  * @event {HelixActivateEvent} helix-activate - Fired once the current one has moved, so the element already holds it.
  * @event {HelixBeforeActivateEvent} helix-before-activate - Fired before the current entry changes, and cancelable.
@@ -80,10 +80,10 @@ export declare class HelixHeadlessVideoPlaylist extends HelixHeadlessVideoPlayli
     /**
      * The player, whichever tag the consumer slotted.
      *
-     * Asked of the slot rather than matched by tag name: the headless tier's
-     * player is `helix-headless-video` and the styled tier's is `helix-video`, and
-     * a consumer bringing a third under either name should still be driven. What
-     * is needed of it is `src`, which is the whole of the contract.
+     * Asked of the slot rather than matched by tag name: each tier registers its
+     * own player under its own name, and a consumer bringing a third should still
+     * be driven. What is needed of it is `src`, which is the whole of the
+     * contract.
      */
     private get player();
     /** Moves by one, and does nothing at either end. */

package/dist/video-playlist/root/video-playlist-root.js

@@ -7,6 +7,7 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
 import { customElement, HelixElement, MixinControllerActive } from '@fusion.dev/helix-core';
 import { html } from 'lit';
 import { property } from 'lit/decorators.js';
+import { HelixHeadlessVideoPlaylistItem } from '../item/video-playlist-item.js';
 let HelixHeadlessVideoPlaylist = class HelixHeadlessVideoPlaylist extends MixinControllerActive(HelixElement) {
     constructor() {
         super(...arguments);
@@ -34,9 +35,7 @@ let HelixHeadlessVideoPlaylist = class HelixHeadlessVideoPlaylist extends MixinC
         };
     }
     get items() {
-        return [
-            ...this.querySelectorAll('helix-video-playlist-item, helix-headless-video-playlist-item'),
-        ].filter((one) => one.value !== '');
+        return [...this.querySelectorAll('*')].filter((one) => one instanceof HelixHeadlessVideoPlaylistItem && one.value !== '');
     }
     get player() {
         const slot = this.renderRoot?.querySelector('slot[name="player"]');

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@fusion.dev/helix-headless",
   "description": "Headless component tier. Structure, accessibility, state, and form participation. No styling.",
-  "version": "0.53.1",
+  "version": "0.53.3",
   "license": "MIT",
   "author": "Ross Sheppard <ross.sheppard@fusion.dev>",
   "repository": {
@@ -46,14 +46,14 @@
   },
   "bugs": "https://github.com/project-fusion/fusion/issues",
   "dependencies": {
-    "@fusion.dev/helix-core": "0.53.1",
-    "@fusion.dev/helix-icons": "0.53.1",
+    "@fusion.dev/helix-core": "0.53.3",
+    "@fusion.dev/helix-icons": "0.53.3",
     "uqr": "^0.1.3"
   },
   "devDependencies": {
     "@custom-elements-manifest/analyzer": "^0.11.0",
-    "@fusion.dev/helix-cem": "0.53.1",
-    "@fusion.dev/helix-testing": "0.53.1",
+    "@fusion.dev/helix-cem": "0.53.3",
+    "@fusion.dev/helix-testing": "0.53.3",
     "@fusion.dev/scripts": "0.0.0-dev",
     "@happy-dom/global-registrator": "^20.0.0",
     "@lit-labs/ssr": "^4.1.0",