stimeo-ui 0.5.0 → 0.6.0

data/dist/controllers/file_dropzone_controller.js

@@ -12,18 +12,38 @@ var FileDropzoneController = class extends Controller {
   static events = ["change", "reject"];
   /** Selected files paired with their rendered item and any preview objectURL. */
   #entries = [];
+  /** Prevents initial and teardown target callbacks from binding outside controller lifetime. */
+  #connected = false;
   /** Wires file removal as a delegated listener on the list container. */
   connect() {
-    if (this.hasListTarget) this.listTarget.addEventListener("click", this.#onItemClick);
+    this.#connected = true;
+    if (this.hasListTarget) this.#bindList(this.listTarget);
   }
   /** Revokes any outstanding preview URLs so none leaks across navigations. */
   disconnect() {
-    if (this.hasListTarget) this.listTarget.removeEventListener("click", this.#onItemClick);
+    this.#connected = false;
+    for (const list of this.listTargets) list.removeEventListener("click", this.#onItemClick);
     for (const entry of this.#entries) {
       if (entry.url) URL.revokeObjectURL(entry.url);
     }
     this.#entries.length = 0;
   }
+  /** Rebinds removal and restores client-only previews when Turbo replaces the list target. */
+  listTargetConnected(list) {
+    if (!this.#connected) return;
+    this.#bindList(list);
+  }
+  /** Releases only the list target that actually disconnected. */
+  listTargetDisconnected(list) {
+    list.removeEventListener("click", this.#onItemClick);
+  }
+  /** Binds delegated removal once and moves live preview items into the current list. */
+  #bindList(list) {
+    list.addEventListener("click", this.#onItemClick);
+    for (const entry of this.#entries) {
+      if (!list.contains(entry.item)) list.appendChild(entry.item);
+    }
+  }
   /** Opens the native file dialog. Bound via `data-action` (trigger click). */
   openDialog() {
     this.inputTarget.click();
@@ -55,8 +75,11 @@ var FileDropzoneController = class extends Controller {
    * an item is appended without waiting on Stimulus to wire a freshly created element.
    */
   #onItemClick = (event) => {
+    const list = event.currentTarget;
     const button = event.target.closest("button");
-    if (!button || !this.hasListTarget || !this.listTarget.contains(button)) return;
+    if (!this.#connected || !list || !button || !this.hasListTarget || list !== this.listTarget || !list.contains(button)) {
+      return;
+    }
     const index = this.#entries.findIndex((entry) => entry.item.contains(button));
     if (index !== -1) this.#removeAt(index);
   };

data/dist/controllers/idle_controller.js

@@ -55,15 +55,37 @@ var SafeTimeout = class extends TimerRegistry {
   }
 };
 
+// src/utils/string_list.ts
+function parseStringList(raw, fallback = []) {
+  const text = raw.trim();
+  if (text.length === 0) return [...fallback];
+  let parsed;
+  try {
+    parsed = JSON.parse(text);
+  } catch {
+    return [...fallback];
+  }
+  if (!Array.isArray(parsed)) return [...fallback];
+  return parsed.filter((entry) => typeof entry === "string");
+}
+
 // src/controllers/idle_controller.ts
+var DEFAULT_ACTIVITY_EVENTS = [
+  "mousemove",
+  "mousedown",
+  "keydown",
+  "wheel",
+  "touchstart",
+  "scroll"
+];
 var IdleController = class extends Controller {
   static values = {
     timeout: { type: Number, default: 9e5 },
     promptBefore: { type: Number, default: 0 },
-    events: {
-      type: Array,
-      default: ["mousemove", "mousedown", "keydown", "wheel", "touchstart", "scroll"]
-    }
+    // A JSON list read through `parseStringList` rather than Stimulus's `Array`
+    // type: that reader throws out of the value observer before any callback
+    // runs, so one malformed attribute would stop the detector connecting.
+    events: { type: String, default: "" }
   };
   static events = ["prompt", "idle", "active"];
   #timeouts = new SafeTimeout();
@@ -94,7 +116,7 @@ var IdleController = class extends Controller {
     this.#idle = false;
     this.#prompted = false;
     this.element.removeAttribute("data-idle");
-    this.#boundEvents = [...this.eventsValue];
+    this.#boundEvents = parseStringList(this.eventsValue, DEFAULT_ACTIVITY_EVENTS);
     for (const type of this.#boundEvents) {
       document.addEventListener(type, this.#onActivity, { passive: true, capture: true });
     }

data/dist/controllers/menubar_controller.js

@@ -125,10 +125,12 @@ var RovingTabindex = class {
    * "nothing is currently tabbable".
    *
    * @param index - Position of the item to make tabbable.
-   * @param options - Pass `{ focus: true }` to also move DOM focus to that item.
+   * @param options - Pass `{ focus: true }` to also move DOM focus to that item,
+   *   and `items` to reuse an event-scoped collection snapshot.
    */
-  setActive(index, { focus = false } = {}) {
-    const items = this.#getItems();
+  setActive(index, options = {}) {
+    const { focus = false } = options;
+    const items = options.items ?? this.#getItems();
     items.forEach((item, i) => {
       item.tabIndex = i === index ? 0 : -1;
     });