@y14e/roving-tabindex 3.0.1 → 3.0.2

package/README.md

@@ -10,14 +10,14 @@ npm i @y14e/roving-tabindex
 
 ```ts
 // npm
-import { createRovingTabIndex } from '@y14e/roving-tabindex@3.0.1';
+import { createRovingTabIndex } from '@y14e/roving-tabindex@3.0.2';
 
 // CDNs
-import { createRovingTabIndex } from 'https://esm.sh/@y14e/roving-tabindex@3.0.1';
+import { createRovingTabIndex } from 'https://esm.sh/@y14e/roving-tabindex@3.0.2';
 // or
-import { createRovingTabIndex } from 'https://cdn.jsdelivr.net/npm/@y14e/roving-tabindex@3.0.1/+esm';
+import { createRovingTabIndex } from 'https://cdn.jsdelivr.net/npm/@y14e/roving-tabindex@3.0.2/+esm';
 // or
-import { createRovingTabIndex } from 'https://esm.unpkg.com/@y14e/roving-tabindex@3.0.1';
+import { createRovingTabIndex } from 'https://esm.unpkg.com/@y14e/roving-tabindex@3.0.2';
 ```
 
 ## 📦 APIs

package/dist/index.cjs

@@ -1,13 +1,13 @@
 'use strict';
 
 // node_modules/@y14e/attributes-utils/dist/index.js
-var defaultParser = (value) => value.split(/\s+/);
-var defaultSerializer = (tokens) => tokens.join(" ");
+var DEFAULT_PARSER = (value) => value.split(/\s+/);
+var DEFAULT_SERIALIZER = (tokens) => tokens.join(" ");
 function addTokenToAttribute(element, attribute, token, options = {}) {
   const {
     caseInsensitive = false,
-    parse = defaultParser,
-    serialize = defaultSerializer
+    parse = DEFAULT_PARSER,
+    serialize = DEFAULT_SERIALIZER
   } = options;
   const value = element.getAttribute(attribute)?.trim();
   const tokens = value ? parse(value).filter(Boolean) : [];
@@ -17,11 +17,11 @@ function addTokenToAttribute(element, attribute, token, options = {}) {
       tokens.push(token);
       element.setAttribute(attribute, serialize(tokens));
     }
-    return;
+  } else {
+    const set = new Set(tokens);
+    set.add(token);
+    element.setAttribute(attribute, serialize([...set]));
   }
-  const set = new Set(tokens);
-  set.add(token);
-  element.setAttribute(attribute, serialize([...set]));
 }
 var snapshots = /* @__PURE__ */ new WeakMap();
 function restoreAttributes(elements) {
@@ -90,21 +90,16 @@ function getFocusables(container = document.body, options = {}) {
   const elements = [];
   if (composed || include) {
     let traverse2 = function(node) {
-      if (node instanceof Element) {
-        if (isFocusable(node, {
-          skipNegativeTabIndexCheck,
-          skipVisibilityCheck
-        }) || include?.(node)) {
-          elements[elements.length] = node;
-        }
+      if (!(node instanceof Element)) {
+        return;
+      }
+      if (isFocusable(node, { skipNegativeTabIndexCheck, skipVisibilityCheck }) || include?.(node)) {
+        elements[elements.length] = node;
       }
       const children = getComposedChildren(node);
       for (let i = 0, l = children.length; i < l; i++) {
         const child = children[i];
-        if (!child) {
-          continue;
-        }
-        traverse2(child);
+        child && traverse2(child);
       }
     };
     traverse2(container);
@@ -112,10 +107,7 @@ function getFocusables(container = document.body, options = {}) {
     const candidates = container.querySelectorAll(FOCUSABLE_SELECTOR);
     for (let i = 0, l = candidates.length; i < l; i++) {
       const candidate = candidates[i];
-      if (!(candidate instanceof Element)) {
-        continue;
-      }
-      if (isFocusable(candidate, {
+      if (candidate && isFocusable(candidate, {
         skipNegativeTabIndexCheck,
         skipVisibilityCheck
       })) {
@@ -218,23 +210,19 @@ function normalizeRadioGroup(elements) {
   for (const group of map.values()) {
     placeholder.add(group.find((radio) => radio.checked) ?? group[0]);
   }
-  return elements.filter((element) => {
-    if (isUngroupedRadio(element)) {
-      return placeholder.has(element);
-    }
-    return true;
-  });
+  return elements.filter(
+    (element) => isUngroupedRadio(element) ? placeholder.has(element) : true
+  );
 }
 function sortByTabIndex(elements) {
   const ordered = [];
   const natural = [];
   for (let i = 0, l = elements.length; i < l; i++) {
     const element = elements[i];
-    if (!element) {
-      continue;
+    if (element) {
+      const target = getTabIndex(element) > 0 ? ordered : natural;
+      target[target.length] = element;
     }
-    const target = getTabIndex(element) > 0 ? ordered : natural;
-    target[target.length] = element;
   }
   ordered.sort((a, b) => getTabIndex(a) - getTabIndex(b));
   let count = 0;
@@ -393,11 +381,7 @@ var RovingTabIndex = class {
       return;
     }
     const isFocusable2 = this.#focusables.has(target);
-    if (this.#options.noMemory && !isFocusable2) {
-      this.#update(null);
-      return;
-    }
-    isFocusable2 && this.#update(target);
+    this.#options.noMemory && !isFocusable2 ? this.#update(null) : isFocusable2 && this.#update(target);
   };
   #onKeyDown = (event) => {
     if (!event.composedPath().includes(this.#container)) {
@@ -465,15 +449,14 @@ var RovingTabIndex = class {
   #update(active) {
     const current = new Set(this.#getFocusables());
     for (const focusable of this.#focusables) {
-      if (current.has(focusable)) {
-        continue;
+      if (!current.has(focusable)) {
+        focusable.isConnected && restoreAttributes([focusable]);
+        this.#focusables.delete(focusable);
+        this.#focusablesByFirstChar.forEach((focusables) => {
+          const index = focusables.indexOf(focusable);
+          index >= 0 && focusables.splice(index, 1);
+        });
       }
-      focusable.isConnected && restoreAttributes([focusable]);
-      this.#focusables.delete(focusable);
-      this.#focusablesByFirstChar.forEach((focusables) => {
-        const index = focusables.indexOf(focusable);
-        index >= 0 && focusables.splice(index, 1);
-      });
     }
     const { navigationOnly, noStart, typeahead } = this.#options;
     for (const focusable of current) {
@@ -506,18 +489,17 @@ var RovingTabIndex = class {
         this.#focusablesByFirstChar.set(key, focusables);
       });
     }
-    if (navigationOnly) {
-      return;
-    }
-    if (active && this.#focusables.has(active)) {
-      this.#focusables.forEach((focusable) => {
-        focusable.setAttribute("tabindex", focusable === active ? "0" : "-1");
-      });
-      return;
+    if (!navigationOnly) {
+      if (active && this.#focusables.has(active)) {
+        this.#focusables.forEach((focusable) => {
+          focusable.setAttribute("tabindex", focusable === active ? "0" : "-1");
+        });
+      } else {
+        [...this.#focusables].forEach((focusable, i) => {
+          focusable.setAttribute("tabindex", i || noStart ? "-1" : "0");
+        });
+      }
     }
-    [...this.#focusables].forEach((focusable, i) => {
-      focusable.setAttribute("tabindex", i || noStart ? "-1" : "0");
-    });
   }
   #createSelectorFilter() {
     const { selector } = this.#options;
@@ -547,7 +529,7 @@ function getActiveElement() {
  * Lightweight roving tabindex utility with fully focus management.
  * Designed for accessible menus, tabs, toolbars, and composite widgets.
  *
- * @version 3.0.1
+ * @version 3.0.2
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane
@@ -559,7 +541,7 @@ function getActiveElement() {
   (**
    * Attributes Utils
    *
-   * @version 1.1.1
+   * @version 1.1.2
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane
@@ -572,7 +554,7 @@ power-focusable/dist/index.js:
    * High-precision focus management utility with full composed tree support.
    * Handles complex focus rules including tabindex ordering, radio groups, inert.
    *
-   * @version 4.3.2
+   * @version 4.3.3
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.d.cts

@@ -3,7 +3,7 @@
  * Lightweight roving tabindex utility with fully focus management.
  * Designed for accessible menus, tabs, toolbars, and composite widgets.
  *
- * @version 3.0.1
+ * @version 3.0.2
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.d.ts

@@ -3,7 +3,7 @@
  * Lightweight roving tabindex utility with fully focus management.
  * Designed for accessible menus, tabs, toolbars, and composite widgets.
  *
- * @version 3.0.1
+ * @version 3.0.2
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.js

@@ -1,11 +1,11 @@
 // node_modules/@y14e/attributes-utils/dist/index.js
-var defaultParser = (value) => value.split(/\s+/);
-var defaultSerializer = (tokens) => tokens.join(" ");
+var DEFAULT_PARSER = (value) => value.split(/\s+/);
+var DEFAULT_SERIALIZER = (tokens) => tokens.join(" ");
 function addTokenToAttribute(element, attribute, token, options = {}) {
   const {
     caseInsensitive = false,
-    parse = defaultParser,
-    serialize = defaultSerializer
+    parse = DEFAULT_PARSER,
+    serialize = DEFAULT_SERIALIZER
   } = options;
   const value = element.getAttribute(attribute)?.trim();
   const tokens = value ? parse(value).filter(Boolean) : [];
@@ -15,11 +15,11 @@ function addTokenToAttribute(element, attribute, token, options = {}) {
       tokens.push(token);
       element.setAttribute(attribute, serialize(tokens));
     }
-    return;
+  } else {
+    const set = new Set(tokens);
+    set.add(token);
+    element.setAttribute(attribute, serialize([...set]));
   }
-  const set = new Set(tokens);
-  set.add(token);
-  element.setAttribute(attribute, serialize([...set]));
 }
 var snapshots = /* @__PURE__ */ new WeakMap();
 function restoreAttributes(elements) {
@@ -88,21 +88,16 @@ function getFocusables(container = document.body, options = {}) {
   const elements = [];
   if (composed || include) {
     let traverse2 = function(node) {
-      if (node instanceof Element) {
-        if (isFocusable(node, {
-          skipNegativeTabIndexCheck,
-          skipVisibilityCheck
-        }) || include?.(node)) {
-          elements[elements.length] = node;
-        }
+      if (!(node instanceof Element)) {
+        return;
+      }
+      if (isFocusable(node, { skipNegativeTabIndexCheck, skipVisibilityCheck }) || include?.(node)) {
+        elements[elements.length] = node;
       }
       const children = getComposedChildren(node);
       for (let i = 0, l = children.length; i < l; i++) {
         const child = children[i];
-        if (!child) {
-          continue;
-        }
-        traverse2(child);
+        child && traverse2(child);
       }
     };
     traverse2(container);
@@ -110,10 +105,7 @@ function getFocusables(container = document.body, options = {}) {
     const candidates = container.querySelectorAll(FOCUSABLE_SELECTOR);
     for (let i = 0, l = candidates.length; i < l; i++) {
       const candidate = candidates[i];
-      if (!(candidate instanceof Element)) {
-        continue;
-      }
-      if (isFocusable(candidate, {
+      if (candidate && isFocusable(candidate, {
         skipNegativeTabIndexCheck,
         skipVisibilityCheck
       })) {
@@ -216,23 +208,19 @@ function normalizeRadioGroup(elements) {
   for (const group of map.values()) {
     placeholder.add(group.find((radio) => radio.checked) ?? group[0]);
   }
-  return elements.filter((element) => {
-    if (isUngroupedRadio(element)) {
-      return placeholder.has(element);
-    }
-    return true;
-  });
+  return elements.filter(
+    (element) => isUngroupedRadio(element) ? placeholder.has(element) : true
+  );
 }
 function sortByTabIndex(elements) {
   const ordered = [];
   const natural = [];
   for (let i = 0, l = elements.length; i < l; i++) {
     const element = elements[i];
-    if (!element) {
-      continue;
+    if (element) {
+      const target = getTabIndex(element) > 0 ? ordered : natural;
+      target[target.length] = element;
     }
-    const target = getTabIndex(element) > 0 ? ordered : natural;
-    target[target.length] = element;
   }
   ordered.sort((a, b) => getTabIndex(a) - getTabIndex(b));
   let count = 0;
@@ -391,11 +379,7 @@ var RovingTabIndex = class {
       return;
     }
     const isFocusable2 = this.#focusables.has(target);
-    if (this.#options.noMemory && !isFocusable2) {
-      this.#update(null);
-      return;
-    }
-    isFocusable2 && this.#update(target);
+    this.#options.noMemory && !isFocusable2 ? this.#update(null) : isFocusable2 && this.#update(target);
   };
   #onKeyDown = (event) => {
     if (!event.composedPath().includes(this.#container)) {
@@ -463,15 +447,14 @@ var RovingTabIndex = class {
   #update(active) {
     const current = new Set(this.#getFocusables());
     for (const focusable of this.#focusables) {
-      if (current.has(focusable)) {
-        continue;
+      if (!current.has(focusable)) {
+        focusable.isConnected && restoreAttributes([focusable]);
+        this.#focusables.delete(focusable);
+        this.#focusablesByFirstChar.forEach((focusables) => {
+          const index = focusables.indexOf(focusable);
+          index >= 0 && focusables.splice(index, 1);
+        });
       }
-      focusable.isConnected && restoreAttributes([focusable]);
-      this.#focusables.delete(focusable);
-      this.#focusablesByFirstChar.forEach((focusables) => {
-        const index = focusables.indexOf(focusable);
-        index >= 0 && focusables.splice(index, 1);
-      });
     }
     const { navigationOnly, noStart, typeahead } = this.#options;
     for (const focusable of current) {
@@ -504,18 +487,17 @@ var RovingTabIndex = class {
         this.#focusablesByFirstChar.set(key, focusables);
       });
     }
-    if (navigationOnly) {
-      return;
-    }
-    if (active && this.#focusables.has(active)) {
-      this.#focusables.forEach((focusable) => {
-        focusable.setAttribute("tabindex", focusable === active ? "0" : "-1");
-      });
-      return;
+    if (!navigationOnly) {
+      if (active && this.#focusables.has(active)) {
+        this.#focusables.forEach((focusable) => {
+          focusable.setAttribute("tabindex", focusable === active ? "0" : "-1");
+        });
+      } else {
+        [...this.#focusables].forEach((focusable, i) => {
+          focusable.setAttribute("tabindex", i || noStart ? "-1" : "0");
+        });
+      }
     }
-    [...this.#focusables].forEach((focusable, i) => {
-      focusable.setAttribute("tabindex", i || noStart ? "-1" : "0");
-    });
   }
   #createSelectorFilter() {
     const { selector } = this.#options;
@@ -545,7 +527,7 @@ function getActiveElement() {
  * Lightweight roving tabindex utility with fully focus management.
  * Designed for accessible menus, tabs, toolbars, and composite widgets.
  *
- * @version 3.0.1
+ * @version 3.0.2
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane
@@ -557,7 +539,7 @@ function getActiveElement() {
   (**
    * Attributes Utils
    *
-   * @version 1.1.1
+   * @version 1.1.2
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane
@@ -570,7 +552,7 @@ power-focusable/dist/index.js:
    * High-precision focus management utility with full composed tree support.
    * Handles complex focus rules including tabindex ordering, radio groups, inert.
    *
-   * @version 4.3.2
+   * @version 4.3.3
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@y14e/roving-tabindex",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "Lightweight roving tabindex utility with fully focus management",
   "type": "module",
   "main": "./dist/index.cjs",
@@ -48,9 +48,9 @@
   },
   "homepage": "https://github.com/y14e/roving-tabindex#readme",
   "devDependencies": {
-    "@y14e/attributes-utils": "^1.1.1",
+    "@y14e/attributes-utils": "^1.1.2",
     "bun-types": "latest",
-    "power-focusable": "^4.3.2",
+    "power-focusable": "^4.3.3",
     "tsup": "^8.0.0",
     "typescript": "^5.6.0"
   },