@f8team/reel-core 1.0.5 → 1.1.1

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @f8team/reel-core
 
+## 1.1.1
+
+### Patch Changes
+
+- e6d5243: Stop a replaced source from loading anyway.
+
+  The YouTube and HLS loaders cleared their `aborted` flag in `detach()`, so a source switch
+  (abort, then detach) let an attach that was still waiting for the YouTube or hls.js runtime carry
+  on: YouTube mounted a second iframe that stayed in the page, and HLS bound a stale hls.js instance
+  to the `<video>`. Each attach now remembers which abort/detach generation it started in and stops
+  when a newer one has happened.
+
+  `<Root>` (React) and `<reel-player>` (Lit) no longer call `setSource` again for the source the
+  player was created with in `options.source`. That second call aborted the first load on mount
+  (an extra media request) and was how the duplicate YouTube iframe was reached.
+
+## 1.1.0
+
+### Minor Changes
+
+- fe0431b: Fix controls that never auto-hide in fullscreen.
+
+  Entering fullscreen is only possible by hitting the fullscreen control, and the
+  pointer focus it leaves behind kept `:focus-within` matching forever, so the
+  themes held the controls bar and the mobile transport at full opacity for the
+  rest of the session. Reported on Android Chrome and Safari.
+
+  The idle timer now releases that pointer focus instead of only dropping
+  `data-reel-controls-visible`. Keyboard focus and open control menus still keep
+  the chrome on screen - the timer re-arms rather than hiding.
+
+  `@f8team/reel-core` gains `blurFocusInside` and `shouldKeepControlsVisible` so
+  the Lit element, the React adapter and app-level players share one auto-hide
+  focus contract; `@f8team/reel-react` re-exports both.
+
 ## 1.0.5
 
 ### Patch Changes

package/dist/index.cjs

@@ -400,6 +400,12 @@ var HlsLoader = class {
     this.rejectAttach = null;
     this.manifestTimeout = null;
     this.aborted = false;
+    /**
+     * Bumped by every abort/detach. `detach()` clears `aborted` so the loader can be reused, so a
+     * source switch (abort, then detach) used to let an attach that was still awaiting the runtime
+     * carry on and bind a stale hls.js instance to the <video>. Each attach keeps its own value.
+     */
+    this.generation = 0;
     this.lastLoadingFragmentKey = null;
     this.repeatedFragmentLoadCount = 0;
     this.repeatedFragmentErrorFired = false;
@@ -418,11 +424,12 @@ var HlsLoader = class {
   async attach(video, source) {
     this.detach();
     this.aborted = false;
+    const generation = this.generation;
     this.video = video;
     this.currentSource = source;
     this.unauthorizedFired = false;
     const Hls = await (this.options.loadRuntime ?? defaultLoad)();
-    if (this.aborted) {
+    if (this.aborted || generation !== this.generation) {
       throw makeAbortError();
     }
     if (!Hls.isSupported()) {
@@ -733,6 +740,7 @@ var HlsLoader = class {
     this.abortLiveXhrs();
   }
   abort() {
+    this.generation += 1;
     if (this.aborted) return;
     this.aborted = true;
     this.clearManifestTimeout();
@@ -1066,17 +1074,24 @@ var YouTubeLoader = class {
     this.duration = 0;
     this.rejectAttach = null;
     this.aborted = false;
+    /**
+     * Bumped by every abort/detach. `detach()` clears `aborted` so the loader can be reused, so a
+     * source switch (abort, then detach) used to let an attach that was still awaiting the runtime
+     * carry on and mount a second iframe. Each attach keeps the value it started with instead.
+     */
+    this.generation = 0;
   }
   async attach(video, source) {
     this.detach();
     this.aborted = false;
+    const generation = this.generation;
     const id = extractYouTubeId(source.src);
     if (!id) {
       throw new Error(`Cannot extract YouTube videoId from "${source.src}"`);
     }
     this.video = video;
     const YT = await (this.options.loadRuntime ?? defaultLoad2)();
-    if (this.aborted) {
+    if (this.aborted || generation !== this.generation) {
       throw makeYtAbortError();
     }
     const parent = video.parentElement;
@@ -1172,6 +1187,7 @@ var YouTubeLoader = class {
     });
   }
   abort() {
+    this.generation += 1;
     if (this.aborted) return;
     this.aborted = true;
     const reject = this.rejectAttach;
@@ -2394,6 +2410,27 @@ function createFocusTrap(container) {
   return { activate, deactivate };
 }
 
+// src/a11y/controlsFocus.ts
+var OPEN_MENU_SELECTOR = "[data-reel-menu-open]";
+function activeElementInside(container) {
+  const active = (container.ownerDocument ?? document).activeElement;
+  if (!(active instanceof HTMLElement)) return null;
+  return container.contains(active) ? active : null;
+}
+function blurFocusInside(container) {
+  activeElementInside(container)?.blur();
+}
+function shouldKeepControlsVisible(container) {
+  if (container.querySelector(OPEN_MENU_SELECTOR) !== null) return true;
+  const active = activeElementInside(container);
+  if (!active) return false;
+  try {
+    return active.matches(":focus-visible");
+  } catch {
+    return false;
+  }
+}
+
 // src/util/time.ts
 var TIME_RE = /^(?:(\d+):)?(\d{1,2}):(\d{1,2})(?:\.(\d+))?$/;
 function parseTime(input) {
@@ -2433,6 +2470,7 @@ exports.TOKEN_TO_CSS_VAR = TOKEN_TO_CSS_VAR;
 exports.announce = announce;
 exports.applyTheme = applyTheme;
 exports.areSourceDescriptorsEquivalent = areSourceDescriptorsEquivalent;
+exports.blurFocusInside = blurFocusInside;
 exports.createFocusTrap = createFocusTrap;
 exports.createPlayer = createPlayer;
 exports.definePlugin = definePlugin;
@@ -2444,6 +2482,7 @@ exports.looksLikeProgressive = looksLikeProgressive;
 exports.looksLikeYouTube = looksLikeYouTube;
 exports.parseTime = parseTime;
 exports.resolveTokens = resolveTokens;
+exports.shouldKeepControlsVisible = shouldKeepControlsVisible;
 exports.toSeconds = toSeconds;
 //# sourceMappingURL=index.cjs.map
 //# sourceMappingURL=index.cjs.map