@fusion.dev/helix-headless 0.59.0 → 0.59.1

package/custom-elements.json

@@ -107645,7 +107645,8 @@
                 "type": {
                   "text": "Promise<void>"
                 }
-              }
+              },
+              "description": "Into picture in picture and back, and quiet about being refused.\n\nCaught rather than `void`ed, the way `FullscreenController` catches its\nown: this is called from a click handler that discards the promise, so a\nrefusal is otherwise an unhandled rejection naming neither the element nor\nthe reason. The roster already declines to draw the button where the\nconsumer refused the feature, which leaves the refusals nothing can see\ncoming, a request without user activation and a window another document is\nalready using."
             },
             {
               "kind": "field",

package/dist/object-inspector/root/object-inspector-root.js

@@ -5,7 +5,7 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
 var HelixHeadlessObjectInspector_1;
-import { CopyController, customElement, HelixElement, inlineStep, scrollItemIntoView, VirtualizeController, } from '@fusion.dev/helix-core';
+import { CopyController, customElement, elementById, HelixElement, inlineStep, scrollItemIntoView, searchableById, VirtualizeController, } from '@fusion.dev/helix-core';
 import { caretDown, caretRight } from '@fusion.dev/helix-icons';
 import { html, nothing } from 'lit';
 import { property, state } from 'lit/decorators.js';
@@ -146,9 +146,7 @@ let HelixHeadlessObjectInspector = class HelixHeadlessObjectInspector extends He
     }
     willUpdate(changed) {
         super.willUpdate(changed);
-        if (this.loadedFor !== this.for &&
-            typeof this.ownerDocument?.getElementById === 'function' &&
-            !this.hydrating) {
+        if (this.loadedFor !== this.for && searchableById(this) && !this.hydrating) {
             this.loadedFor = this.for;
             this.load();
         }
@@ -178,7 +176,7 @@ let HelixHeadlessObjectInspector = class HelixHeadlessObjectInspector extends He
         }
     }
     load() {
-        const text = this.for === '' ? '' : (this.ownerDocument.getElementById(this.for)?.textContent ?? '');
+        const text = this.for === '' ? '' : (elementById(this, this.for)?.textContent ?? '');
         try {
             this.read = text.trim() === '' ? undefined : JSON.parse(text);
         }

package/dist/video/root/extra-controls.d.ts

@@ -13,6 +13,8 @@ export interface Conditions {
     hydrating: boolean;
     /** Whether anything on the network has offered to play this. */
     remote: boolean;
+    /** Whether the consumer has refused picture in picture on the element. */
+    refusedPip: boolean;
     /** Whether a file is named, which is what a download needs. */
     source: boolean;
 }

package/dist/video/root/extra-controls.js

@@ -1,7 +1,10 @@
 import { writtenList } from '@fusion.dev/helix-core';
 export const offers = (one, player) => {
     if (one === 'pip') {
-        return !player.hydrating && typeof document !== 'undefined' && document.pictureInPictureEnabled;
+        return (!player.hydrating &&
+            !player.refusedPip &&
+            typeof document !== 'undefined' &&
+            document.pictureInPictureEnabled);
     }
     if (one === 'remote') {
         return player.remote;

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

@@ -510,6 +510,17 @@ export declare class HelixHeadlessVideo extends HelixElement {
     private skip;
     /** Steps through the speeds offered, which is what one button can do. */
     private cycleSpeed;
+    /**
+     * Into picture in picture and back, and quiet about being refused.
+     *
+     * Caught rather than `void`ed, the way `FullscreenController` catches its
+     * own: this is called from a click handler that discards the promise, so a
+     * refusal is otherwise an unhandled rejection naming neither the element nor
+     * the reason. The roster already declines to draw the button where the
+     * consumer refused the feature, which leaves the refusals nothing can see
+     * coming, a request without user activation and a window another document is
+     * already using.
+     */
     private togglePip;
     /**
      * A press on the picture plays or pauses it.

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

@@ -174,7 +174,12 @@ let HelixHeadlessVideo = class HelixHeadlessVideo extends HelixElement {
         this.captions = [];
     }
     get conditions() {
-        return { hydrating: this.hydrating, remote: this.remoteAvailable, source: this.src !== '' };
+        return {
+            hydrating: this.hydrating,
+            refusedPip: this.disablePictureInPicture,
+            remote: this.remoteAvailable,
+            source: this.src !== '',
+        };
     }
     get extras() {
         return barControls(this.extraControls, this.conditions);
@@ -347,11 +352,16 @@ let HelixHeadlessVideo = class HelixHeadlessVideo extends HelixElement {
         if (!video) {
             return;
         }
-        if (document.pictureInPictureElement === video) {
-            await document.exitPictureInPicture();
-            return;
+        try {
+            if (document.pictureInPictureElement === video) {
+                await document.exitPictureInPicture();
+                return;
+            }
+            await video.requestPictureInPicture();
+        }
+        catch (error) {
+            console.warn(`<${this.localName}> could not enter picture in picture. A press has to come from the reader, and one window holds it at a time.`, error);
         }
-        await video.requestPictureInPicture();
     }
     willUpdate(changed) {
         super.willUpdate(changed);

package/dist/virtual-list/root/virtual-list-root.d.ts

@@ -83,9 +83,15 @@ export declare class HelixHeadlessVirtualList extends HelixElement {
     /**
      * Reads the source named by `for`, before the render that draws it.
      *
+     * **Resolved through `getRootNode()`, so a list inside another component's
+     * shadow root names a source in that root rather than in the page.** Read
+     * against `ownerDocument` it looked in the document whatever tree it was in,
+     * and found nothing from inside an app shell or a sandbox, which is
+     * indistinguishable here from the miss this tolerates on purpose.
+     *
      * Not on the server, per the server rules: the shim has attributes and no
-     * tree, so `getElementById` is undefined there. Nor while hydrating what a
-     * server sent, which it drew with nothing read. A source that is missing or
+     * tree to look an id up in. Nor while hydrating what a server sent, which it
+     * drew with nothing read. A source that is missing or
      * not an array leaves the list empty rather than throwing, because a consumer
      * whose data has not arrived yet is the ordinary case and an empty list says
      * so by itself.

package/dist/virtual-list/root/virtual-list-root.js

@@ -4,7 +4,7 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
-import { customElement, HelixElement, VirtualizeController } from '@fusion.dev/helix-core';
+import { customElement, elementById, HelixElement, searchableById, VirtualizeController, } from '@fusion.dev/helix-core';
 import { html } from 'lit';
 import { property, state } from 'lit/decorators.js';
 let HelixHeadlessVirtualList = class HelixHeadlessVirtualList extends HelixElement {
@@ -31,9 +31,7 @@ let HelixHeadlessVirtualList = class HelixHeadlessVirtualList extends HelixEleme
     }
     willUpdate(changed) {
         super.willUpdate(changed);
-        if (this.readFor !== this.for &&
-            typeof this.ownerDocument?.getElementById === 'function' &&
-            !this.hydrating) {
+        if (this.readFor !== this.for && searchableById(this) && !this.hydrating) {
             this.readFor = this.for;
             this.read();
         }
@@ -48,7 +46,7 @@ let HelixHeadlessVirtualList = class HelixHeadlessVirtualList extends HelixEleme
         this.place();
     }
     read() {
-        const source = this.for === '' ? null : this.ownerDocument.getElementById(this.for);
+        const source = this.for === '' ? null : elementById(this, this.for);
         const text = source?.textContent ?? '';
         if (text.trim() === '') {
             this.rows = [];

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.59.0",
+  "version": "0.59.1",
   "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.59.0",
-    "@fusion.dev/helix-icons": "0.59.0",
+    "@fusion.dev/helix-core": "0.59.1",
+    "@fusion.dev/helix-icons": "0.59.1",
     "uqr": "^0.1.3"
   },
   "devDependencies": {
     "@custom-elements-manifest/analyzer": "^0.11.0",
-    "@fusion.dev/helix-cem": "0.59.0",
-    "@fusion.dev/helix-testing": "0.59.0",
+    "@fusion.dev/helix-cem": "0.59.1",
+    "@fusion.dev/helix-testing": "0.59.1",
     "@fusion.dev/scripts": "0.0.0-dev",
     "@happy-dom/global-registrator": "^20.0.0",
     "@lit-labs/ssr": "^4.1.0",