@y14e/accordion 1.4.10 → 1.4.12

package/README.md

@@ -10,14 +10,14 @@ npm i @y14e/accordion
 
 ```ts
 // npm
-import Accordion from '@y14e/accordion@1.4.10';
+import Accordion from '@y14e/accordion@1.4.12';
 
 // CDNs
-import Accordion from 'https://esm.sh/@y14e/accordion@1.4.10';
+import Accordion from 'https://esm.sh/@y14e/accordion@1.4.12';
 // or
-import Accordion from 'https://cdn.jsdelivr.net/npm/@y14e/accordion@1.4.10/+esm';
+import Accordion from 'https://cdn.jsdelivr.net/npm/@y14e/accordion@1.4.12/+esm';
 // or
-import Accordion from 'https://esm.unpkg.com/@y14e/accordion@1.4.10';
+import Accordion from 'https://esm.unpkg.com/@y14e/accordion@1.4.12';
 ```
 
 ## Usage

package/dist/index.cjs

@@ -90,10 +90,11 @@ var Button = class {
       return;
     }
     const active = getActiveElement();
-    if (active instanceof HTMLElement) {
-      event.preventDefault();
-      active.click();
+    if (!(active instanceof HTMLElement)) {
+      return;
     }
+    event.preventDefault();
+    active.click();
   };
 };
 function getActiveElement() {
@@ -477,10 +478,11 @@ var RovingTabIndex = class {
   }
   #onFocusIn = (event) => {
     const { target } = event;
-    if (target instanceof Element) {
-      const isFocusable22 = this.#focusables.has(target);
-      this.#options.noMemory && !isFocusable22 ? this.#update(null) : isFocusable22 && this.#update(target);
+    if (!(target instanceof Element)) {
+      return;
     }
+    const isFocusable22 = this.#focusables.has(target);
+    this.#options.noMemory && !isFocusable22 ? this.#update(null) : isFocusable22 && this.#update(target);
   };
   #onKeyDown = (event) => {
     if (!event.composedPath().includes(this.#container)) {
@@ -678,11 +680,12 @@ var Accordion = class _Accordion {
     }
     this.#triggerElements.forEach((trigger2, i) => {
       const content2 = this.#contentElements[i];
-      if (content2) {
-        const binding = createBinding(trigger2, content2);
-        this.#bindings.set(trigger2, binding);
-        this.#bindings.set(content2, binding);
+      if (!content2) {
+        return;
       }
+      const binding = createBinding(trigger2, content2);
+      this.#bindings.set(trigger2, binding);
+      this.#bindings.set(content2, binding);
     });
     this.#initialize();
   }
@@ -690,11 +693,11 @@ var Accordion = class _Accordion {
     if (this.#isDestroyed) {
       return;
     }
-    if (trigger instanceof HTMLElement && this.#bindings.has(trigger)) {
-      this.#toggle(trigger, false);
-    } else {
+    if (!(trigger instanceof HTMLElement) || !this.#bindings.has(trigger)) {
       console.warn("Invalid trigger element");
+      return;
     }
+    this.#toggle(trigger, false);
   }
   async destroy(force = false) {
     if (this.#isDestroyed) {
@@ -725,11 +728,11 @@ var Accordion = class _Accordion {
     if (this.#isDestroyed) {
       return;
     }
-    if (trigger instanceof HTMLElement && this.#bindings.has(trigger)) {
-      this.#toggle(trigger, true);
-    } else {
+    if (!(trigger instanceof HTMLElement) || !this.#bindings.has(trigger)) {
       console.warn("Invalid trigger element");
+      return;
     }
+    this.#toggle(trigger, true);
   }
   #initialize() {
     saveAttributes(this.#triggerElements, [
@@ -780,16 +783,21 @@ var Accordion = class _Accordion {
   #onTriggerClick = (event) => {
     event.preventDefault();
     const trigger = event.currentTarget;
-    if (trigger instanceof HTMLElement) {
-      this.#toggle(trigger, trigger.ariaExpanded === "false");
+    if (!(trigger instanceof HTMLElement)) {
+      return;
     }
+    this.#toggle(trigger, trigger.ariaExpanded === "false");
   };
   #onContentBeforeMatch = (event) => {
     const content = event.currentTarget;
-    if (content instanceof HTMLElement) {
-      const binding = this.#bindings.get(content);
-      binding && binding.trigger.ariaExpanded === "false" && this.#toggle(binding.trigger, true, true);
+    if (!(content instanceof HTMLElement)) {
+      return;
     }
+    const binding = this.#bindings.get(content);
+    if (!binding) {
+      return;
+    }
+    binding.trigger.ariaExpanded === "false" && this.#toggle(binding.trigger, true, true);
   };
   #toggle(trigger, isOpen, isMatch = false) {
     if (trigger.ariaExpanded === String(isOpen)) {
@@ -854,7 +862,10 @@ var Accordion = class _Accordion {
   }
   #onAnimationFinish(content) {
     const trigger = this.#bindings.get(content)?.trigger;
-    if (trigger && trigger.ariaExpanded === "false") {
+    if (!trigger) {
+      return;
+    }
+    if (trigger.ariaExpanded === "false") {
       content.setAttribute("hidden", "until-found");
     }
     ["block-size", "overflow"].forEach((name) => {
@@ -889,7 +900,7 @@ function waitAnimationFinish(animation) {
  * Accordion
  * WAI-ARIA compliant accordion pattern implementation in TypeScript.
  *
- * @version 1.4.10
+ * @version 1.4.12
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane
@@ -912,7 +923,7 @@ function waitAnimationFinish(animation) {
   (**
    * Button
    *
-   * @version 1.0.1
+   * @version 1.0.2
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane
@@ -925,7 +936,7 @@ function waitAnimationFinish(animation) {
    * Lightweight roving tabindex utility with fully focus management.
    * Designed for accessible menus, tabs, toolbars, and composite widgets.
    *
-   * @version 3.0.4
+   * @version 3.0.5
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.d.cts

@@ -2,7 +2,7 @@
  * Accordion
  * WAI-ARIA compliant accordion pattern implementation in TypeScript.
  *
- * @version 1.4.10
+ * @version 1.4.12
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.d.ts

@@ -2,7 +2,7 @@
  * Accordion
  * WAI-ARIA compliant accordion pattern implementation in TypeScript.
  *
- * @version 1.4.10
+ * @version 1.4.12
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane

package/dist/index.js

@@ -88,10 +88,11 @@ var Button = class {
       return;
     }
     const active = getActiveElement();
-    if (active instanceof HTMLElement) {
-      event.preventDefault();
-      active.click();
+    if (!(active instanceof HTMLElement)) {
+      return;
     }
+    event.preventDefault();
+    active.click();
   };
 };
 function getActiveElement() {
@@ -475,10 +476,11 @@ var RovingTabIndex = class {
   }
   #onFocusIn = (event) => {
     const { target } = event;
-    if (target instanceof Element) {
-      const isFocusable22 = this.#focusables.has(target);
-      this.#options.noMemory && !isFocusable22 ? this.#update(null) : isFocusable22 && this.#update(target);
+    if (!(target instanceof Element)) {
+      return;
     }
+    const isFocusable22 = this.#focusables.has(target);
+    this.#options.noMemory && !isFocusable22 ? this.#update(null) : isFocusable22 && this.#update(target);
   };
   #onKeyDown = (event) => {
     if (!event.composedPath().includes(this.#container)) {
@@ -676,11 +678,12 @@ var Accordion = class _Accordion {
     }
     this.#triggerElements.forEach((trigger2, i) => {
       const content2 = this.#contentElements[i];
-      if (content2) {
-        const binding = createBinding(trigger2, content2);
-        this.#bindings.set(trigger2, binding);
-        this.#bindings.set(content2, binding);
+      if (!content2) {
+        return;
       }
+      const binding = createBinding(trigger2, content2);
+      this.#bindings.set(trigger2, binding);
+      this.#bindings.set(content2, binding);
     });
     this.#initialize();
   }
@@ -688,11 +691,11 @@ var Accordion = class _Accordion {
     if (this.#isDestroyed) {
       return;
     }
-    if (trigger instanceof HTMLElement && this.#bindings.has(trigger)) {
-      this.#toggle(trigger, false);
-    } else {
+    if (!(trigger instanceof HTMLElement) || !this.#bindings.has(trigger)) {
       console.warn("Invalid trigger element");
+      return;
     }
+    this.#toggle(trigger, false);
   }
   async destroy(force = false) {
     if (this.#isDestroyed) {
@@ -723,11 +726,11 @@ var Accordion = class _Accordion {
     if (this.#isDestroyed) {
       return;
     }
-    if (trigger instanceof HTMLElement && this.#bindings.has(trigger)) {
-      this.#toggle(trigger, true);
-    } else {
+    if (!(trigger instanceof HTMLElement) || !this.#bindings.has(trigger)) {
       console.warn("Invalid trigger element");
+      return;
     }
+    this.#toggle(trigger, true);
   }
   #initialize() {
     saveAttributes(this.#triggerElements, [
@@ -778,16 +781,21 @@ var Accordion = class _Accordion {
   #onTriggerClick = (event) => {
     event.preventDefault();
     const trigger = event.currentTarget;
-    if (trigger instanceof HTMLElement) {
-      this.#toggle(trigger, trigger.ariaExpanded === "false");
+    if (!(trigger instanceof HTMLElement)) {
+      return;
     }
+    this.#toggle(trigger, trigger.ariaExpanded === "false");
   };
   #onContentBeforeMatch = (event) => {
     const content = event.currentTarget;
-    if (content instanceof HTMLElement) {
-      const binding = this.#bindings.get(content);
-      binding && binding.trigger.ariaExpanded === "false" && this.#toggle(binding.trigger, true, true);
+    if (!(content instanceof HTMLElement)) {
+      return;
     }
+    const binding = this.#bindings.get(content);
+    if (!binding) {
+      return;
+    }
+    binding.trigger.ariaExpanded === "false" && this.#toggle(binding.trigger, true, true);
   };
   #toggle(trigger, isOpen, isMatch = false) {
     if (trigger.ariaExpanded === String(isOpen)) {
@@ -852,7 +860,10 @@ var Accordion = class _Accordion {
   }
   #onAnimationFinish(content) {
     const trigger = this.#bindings.get(content)?.trigger;
-    if (trigger && trigger.ariaExpanded === "false") {
+    if (!trigger) {
+      return;
+    }
+    if (trigger.ariaExpanded === "false") {
       content.setAttribute("hidden", "until-found");
     }
     ["block-size", "overflow"].forEach((name) => {
@@ -887,7 +898,7 @@ function waitAnimationFinish(animation) {
  * Accordion
  * WAI-ARIA compliant accordion pattern implementation in TypeScript.
  *
- * @version 1.4.10
+ * @version 1.4.12
  * @author Yusuke Kamiyamane
  * @license MIT
  * @copyright Copyright (c) Yusuke Kamiyamane
@@ -910,7 +921,7 @@ function waitAnimationFinish(animation) {
   (**
    * Button
    *
-   * @version 1.0.1
+   * @version 1.0.2
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane
@@ -923,7 +934,7 @@ function waitAnimationFinish(animation) {
    * Lightweight roving tabindex utility with fully focus management.
    * Designed for accessible menus, tabs, toolbars, and composite widgets.
    *
-   * @version 3.0.4
+   * @version 3.0.5
    * @author Yusuke Kamiyamane
    * @license MIT
    * @copyright Copyright (c) Yusuke Kamiyamane

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@y14e/accordion",
-  "version": "1.4.10",
+  "version": "1.4.12",
   "description": "WAI-ARIA compliant accordion pattern implementation in TypeScript",
   "type": "module",
   "main": "./dist/index.cjs",
@@ -44,8 +44,8 @@
   "homepage": "https://github.com/y14e/accordion#readme",
   "devDependencies": {
     "@y14e/attributes-utils": "^1.1.2",
-    "@y14e/button": "^1.0.1",
-    "@y14e/roving-tabindex": "^3.0.4",
+    "@y14e/button": "^1.0.2",
+    "@y14e/roving-tabindex": "^3.0.5",
     "bun-types": "latest",
     "tsup": "^8.0.0",
     "typescript": "^5.6.0",