@scarlett-player/ui 1.15.1 → 1.15.2

package/dist/index.cjs

@@ -21,6 +21,7 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 var index_exports = {};
 __export(index_exports, {
   DEFAULT_PRIORITY: () => DEFAULT_PRIORITY,
+  UNRESOLVED_SLOT_GRACE_MS: () => UNRESOLVED_SLOT_GRACE_MS,
   assertFitLayout: () => assertFitLayout,
   default: () => index_default,
   formatLiveTime: () => import_core2.formatLiveTime,
@@ -3642,7 +3643,7 @@ function resetControlRegistry() {
 }
 
 // src/version.ts
-var PKG_VERSION = true ? "1.15.0" : "0.0.0-dev";
+var PKG_VERSION = true ? "1.15.2" : "0.0.0-dev";
 
 // src/index.ts
 var DEFAULT_LAYOUT = [
@@ -3683,6 +3684,7 @@ var RESIZE_FALLBACK_DEBOUNCE_MS = 100;
 var MENU_HEIGHT_RESERVE = 16;
 var FALLBACK_BAR_HEIGHT = 56;
 var MIN_MENU_HEIGHT = 120;
+var UNRESOLVED_SLOT_GRACE_MS = 1500;
 function uiPlugin(config = {}) {
   let api;
   let controlBar = null;
@@ -3714,6 +3716,9 @@ function uiPlugin(config = {}) {
   let lastFitSignature = null;
   let fitPending = true;
   let addedContainerClass = false;
+  let rebuildQueued = false;
+  const unresolvedSlots = /* @__PURE__ */ new Set();
+  let unresolvedSlotTimer = null;
   const layout = config.controls || DEFAULT_LAYOUT;
   const hideDelay = config.hideDelay ?? DEFAULT_HIDE_DELAY;
   const showBigPlayButton = config.bigPlayButton !== false;
@@ -3765,15 +3770,26 @@ function uiPlugin(config = {}) {
             return null;
           }
         }
-        api.logger.warn(`Unknown control slot: ${slot}`);
+        unresolvedSlots.add(slot);
+        api.logger.debug(`No control registered for slot "${slot}" yet`);
         return null;
       }
     }
   };
+  const reportUnresolvedSlots = () => {
+    unresolvedSlotTimer = null;
+    for (const slot of unresolvedSlots) {
+      api.logger.warn(
+        `Control slot "${slot}" has no registered control after ${UNRESOLVED_SLOT_GRACE_MS}ms - is the plugin installed?`
+      );
+    }
+    unresolvedSlots.clear();
+  };
   const populateControlBar = () => {
     if (!controlBar) {
       return;
     }
+    unresolvedSlots.clear();
     const rules = new Map(
       resolveFitItems(layout, config.priority).map((template) => [template.id, template])
     );
@@ -3963,6 +3979,20 @@ function uiPlugin(config = {}) {
     populateControlBar();
     updateControls();
   };
+  const flushRebuild = () => {
+    if (!rebuildQueued) {
+      return;
+    }
+    rebuildQueued = false;
+    rebuildControlBar();
+  };
+  const queueRebuild = () => {
+    if (rebuildQueued) {
+      return;
+    }
+    rebuildQueued = true;
+    queueMicrotask(flushRebuild);
+  };
   const updateControls = () => {
     controls.forEach((c) => c.update());
     progressBar?.update();
@@ -4191,6 +4221,9 @@ function uiPlugin(config = {}) {
       controlBar.setAttribute("role", "toolbar");
       controlBar.setAttribute("aria-label", "Video controls");
       populateControlBar();
+      if (unresolvedSlots.size > 0) {
+        unresolvedSlotTimer = setTimeout(reportUnresolvedSlots, UNRESOLVED_SLOT_GRACE_MS);
+      }
       container.appendChild(controlBar);
       if (responsive && typeof ResizeObserver === "function") {
         resizeObserver = new ResizeObserver((observed) => {
@@ -4218,8 +4251,8 @@ function uiPlugin(config = {}) {
         if (owner && owner !== api.container) {
           return;
         }
-        api.logger.debug(`Control "${id}" registered after init, rebuilding control bar`);
-        rebuildControlBar();
+        api.logger.debug(`Control "${id}" registered after init, queuing a control bar rebuild`);
+        queueRebuild();
       });
       container.addEventListener("pointerdown", handlePointerActivity, { passive: true });
       container.addEventListener("pointermove", handlePointerActivity, { passive: true });
@@ -4283,6 +4316,12 @@ function uiPlugin(config = {}) {
       document.removeEventListener("keydown", handleKeyDown);
       controlRegistryUnsubscribe?.();
       controlRegistryUnsubscribe = null;
+      rebuildQueued = false;
+      if (unresolvedSlotTimer) {
+        clearTimeout(unresolvedSlotTimer);
+        unresolvedSlotTimer = null;
+      }
+      unresolvedSlots.clear();
       controls.forEach((c) => c.destroy());
       controls = [];
       entries = [];
@@ -4347,6 +4386,7 @@ var index_default = uiPlugin;
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   DEFAULT_PRIORITY,
+  UNRESOLVED_SLOT_GRACE_MS,
   assertFitLayout,
   formatLiveTime,
   formatTime,

package/dist/index.d.cts

@@ -605,6 +605,23 @@ declare const styles = "\n/* ============================================\n   Co
  * @packageDocumentation
  */
 
+/**
+ * How long a layout slot may stay without a registered control before the
+ * plugin warns about it.
+ *
+ * Plugin init order is not guaranteed, and the plugins that contribute
+ * controls (playlist, chapters) register them from inside
+ * `import('@scarlett-player/ui').then(...)`, which lands at least a microtask
+ * after this plugin has built its bar. Warning inline at build time therefore
+ * flagged every one of those controls on every mount, and again on each
+ * rebuild that followed. The bar is built first and the layout judged after
+ * this grace instead: the ui chunk is normally already loaded so the import
+ * resolves at once, but a host that split it lazily may still be fetching it.
+ * A typo in the layout still warns, just this much later.
+ *
+ * @internal Exported so the tests do not hard-code the number.
+ */
+declare const UNRESOLVED_SLOT_GRACE_MS = 1500;
 /**
  * Create a UI controls plugin instance.
  *
@@ -631,4 +648,4 @@ declare const styles = "\n/* ============================================\n   Co
  */
 declare function uiPlugin(config?: UIPluginConfig): IUIPlugin;
 
-export { type BuiltinControlSlot, type Control, type ControlFactory, type ControlSlot, DEFAULT_PRIORITY, type FitExit, type FitItem, type FitPlan, type FitRank, type FitRule, type FitTemplate, type IUIPlugin, type LayoutConfig, type RegisterControlOptions, type ThemeConfig, type TimelineExtension, type TimelineExtensionFactory, type TimelineSurface, type UIPluginConfig, assertFitLayout, uiPlugin as default, getControlFactory, hasTimelineExtension, icons, planFit, registerControl, registerTimelineExtension, resetControlRegistry, resolveFitItems, styles, uiPlugin, unregisterControl, unregisterControlsFor, unregisterTimelineExtension };
+export { type BuiltinControlSlot, type Control, type ControlFactory, type ControlSlot, DEFAULT_PRIORITY, type FitExit, type FitItem, type FitPlan, type FitRank, type FitRule, type FitTemplate, type IUIPlugin, type LayoutConfig, type RegisterControlOptions, type ThemeConfig, type TimelineExtension, type TimelineExtensionFactory, type TimelineSurface, type UIPluginConfig, UNRESOLVED_SLOT_GRACE_MS, assertFitLayout, uiPlugin as default, getControlFactory, hasTimelineExtension, icons, planFit, registerControl, registerTimelineExtension, resetControlRegistry, resolveFitItems, styles, uiPlugin, unregisterControl, unregisterControlsFor, unregisterTimelineExtension };

package/dist/index.d.ts

@@ -605,6 +605,23 @@ declare const styles = "\n/* ============================================\n   Co
  * @packageDocumentation
  */
 
+/**
+ * How long a layout slot may stay without a registered control before the
+ * plugin warns about it.
+ *
+ * Plugin init order is not guaranteed, and the plugins that contribute
+ * controls (playlist, chapters) register them from inside
+ * `import('@scarlett-player/ui').then(...)`, which lands at least a microtask
+ * after this plugin has built its bar. Warning inline at build time therefore
+ * flagged every one of those controls on every mount, and again on each
+ * rebuild that followed. The bar is built first and the layout judged after
+ * this grace instead: the ui chunk is normally already loaded so the import
+ * resolves at once, but a host that split it lazily may still be fetching it.
+ * A typo in the layout still warns, just this much later.
+ *
+ * @internal Exported so the tests do not hard-code the number.
+ */
+declare const UNRESOLVED_SLOT_GRACE_MS = 1500;
 /**
  * Create a UI controls plugin instance.
  *
@@ -631,4 +648,4 @@ declare const styles = "\n/* ============================================\n   Co
  */
 declare function uiPlugin(config?: UIPluginConfig): IUIPlugin;
 
-export { type BuiltinControlSlot, type Control, type ControlFactory, type ControlSlot, DEFAULT_PRIORITY, type FitExit, type FitItem, type FitPlan, type FitRank, type FitRule, type FitTemplate, type IUIPlugin, type LayoutConfig, type RegisterControlOptions, type ThemeConfig, type TimelineExtension, type TimelineExtensionFactory, type TimelineSurface, type UIPluginConfig, assertFitLayout, uiPlugin as default, getControlFactory, hasTimelineExtension, icons, planFit, registerControl, registerTimelineExtension, resetControlRegistry, resolveFitItems, styles, uiPlugin, unregisterControl, unregisterControlsFor, unregisterTimelineExtension };
+export { type BuiltinControlSlot, type Control, type ControlFactory, type ControlSlot, DEFAULT_PRIORITY, type FitExit, type FitItem, type FitPlan, type FitRank, type FitRule, type FitTemplate, type IUIPlugin, type LayoutConfig, type RegisterControlOptions, type ThemeConfig, type TimelineExtension, type TimelineExtensionFactory, type TimelineSurface, type UIPluginConfig, UNRESOLVED_SLOT_GRACE_MS, assertFitLayout, uiPlugin as default, getControlFactory, hasTimelineExtension, icons, planFit, registerControl, registerTimelineExtension, resetControlRegistry, resolveFitItems, styles, uiPlugin, unregisterControl, unregisterControlsFor, unregisterTimelineExtension };

package/dist/index.js

@@ -3606,7 +3606,7 @@ function resetControlRegistry() {
 }
 
 // src/version.ts
-var PKG_VERSION = true ? "1.15.0" : "0.0.0-dev";
+var PKG_VERSION = true ? "1.15.2" : "0.0.0-dev";
 
 // src/index.ts
 var DEFAULT_LAYOUT = [
@@ -3647,6 +3647,7 @@ var RESIZE_FALLBACK_DEBOUNCE_MS = 100;
 var MENU_HEIGHT_RESERVE = 16;
 var FALLBACK_BAR_HEIGHT = 56;
 var MIN_MENU_HEIGHT = 120;
+var UNRESOLVED_SLOT_GRACE_MS = 1500;
 function uiPlugin(config = {}) {
   let api;
   let controlBar = null;
@@ -3678,6 +3679,9 @@ function uiPlugin(config = {}) {
   let lastFitSignature = null;
   let fitPending = true;
   let addedContainerClass = false;
+  let rebuildQueued = false;
+  const unresolvedSlots = /* @__PURE__ */ new Set();
+  let unresolvedSlotTimer = null;
   const layout = config.controls || DEFAULT_LAYOUT;
   const hideDelay = config.hideDelay ?? DEFAULT_HIDE_DELAY;
   const showBigPlayButton = config.bigPlayButton !== false;
@@ -3729,15 +3733,26 @@ function uiPlugin(config = {}) {
             return null;
           }
         }
-        api.logger.warn(`Unknown control slot: ${slot}`);
+        unresolvedSlots.add(slot);
+        api.logger.debug(`No control registered for slot "${slot}" yet`);
         return null;
       }
     }
   };
+  const reportUnresolvedSlots = () => {
+    unresolvedSlotTimer = null;
+    for (const slot of unresolvedSlots) {
+      api.logger.warn(
+        `Control slot "${slot}" has no registered control after ${UNRESOLVED_SLOT_GRACE_MS}ms - is the plugin installed?`
+      );
+    }
+    unresolvedSlots.clear();
+  };
   const populateControlBar = () => {
     if (!controlBar) {
       return;
     }
+    unresolvedSlots.clear();
     const rules = new Map(
       resolveFitItems(layout, config.priority).map((template) => [template.id, template])
     );
@@ -3927,6 +3942,20 @@ function uiPlugin(config = {}) {
     populateControlBar();
     updateControls();
   };
+  const flushRebuild = () => {
+    if (!rebuildQueued) {
+      return;
+    }
+    rebuildQueued = false;
+    rebuildControlBar();
+  };
+  const queueRebuild = () => {
+    if (rebuildQueued) {
+      return;
+    }
+    rebuildQueued = true;
+    queueMicrotask(flushRebuild);
+  };
   const updateControls = () => {
     controls.forEach((c) => c.update());
     progressBar?.update();
@@ -4155,6 +4184,9 @@ function uiPlugin(config = {}) {
       controlBar.setAttribute("role", "toolbar");
       controlBar.setAttribute("aria-label", "Video controls");
       populateControlBar();
+      if (unresolvedSlots.size > 0) {
+        unresolvedSlotTimer = setTimeout(reportUnresolvedSlots, UNRESOLVED_SLOT_GRACE_MS);
+      }
       container.appendChild(controlBar);
       if (responsive && typeof ResizeObserver === "function") {
         resizeObserver = new ResizeObserver((observed) => {
@@ -4182,8 +4214,8 @@ function uiPlugin(config = {}) {
         if (owner && owner !== api.container) {
           return;
         }
-        api.logger.debug(`Control "${id}" registered after init, rebuilding control bar`);
-        rebuildControlBar();
+        api.logger.debug(`Control "${id}" registered after init, queuing a control bar rebuild`);
+        queueRebuild();
       });
       container.addEventListener("pointerdown", handlePointerActivity, { passive: true });
       container.addEventListener("pointermove", handlePointerActivity, { passive: true });
@@ -4247,6 +4279,12 @@ function uiPlugin(config = {}) {
       document.removeEventListener("keydown", handleKeyDown);
       controlRegistryUnsubscribe?.();
       controlRegistryUnsubscribe = null;
+      rebuildQueued = false;
+      if (unresolvedSlotTimer) {
+        clearTimeout(unresolvedSlotTimer);
+        unresolvedSlotTimer = null;
+      }
+      unresolvedSlots.clear();
       controls.forEach((c) => c.destroy());
       controls = [];
       entries = [];
@@ -4310,6 +4348,7 @@ function uiPlugin(config = {}) {
 var index_default = uiPlugin;
 export {
   DEFAULT_PRIORITY,
+  UNRESOLVED_SLOT_GRACE_MS,
   assertFitLayout,
   index_default as default,
   formatLiveTime,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scarlett-player/ui",
-  "version": "1.15.1",
+  "version": "1.15.2",
   "description": "UI Controls Plugin for Scarlett Player",
   "type": "module",
   "main": "dist/index.cjs",
@@ -29,7 +29,7 @@
     "typescript": "^5.3.0",
     "vitest": "^1.6.0",
     "jsdom": "^24.0.0",
-    "@scarlett-player/core": "1.15.1"
+    "@scarlett-player/core": "1.15.2"
   },
   "keywords": [
     "video",