tweaks

Author: adolgachev

src/aria/accordion/accordion.ts

@@ -235,33 +235,46 @@ export class AccordionGroup {
     });
   }
 
-  /** Opens the specified item. */
+  /** Opens the accordion panel with the specified value. */
   open(itemValue: string) {
-    this._pattern.expansionManager.open(this._patternForValue(itemValue)!);
+    const trigger = this._findTriggerPatternByValue(itemValue);
+
+    if (trigger) {
+      this._pattern.expansionManager.open(trigger);
+    }
   }
 
-  /** Closes the specified item. */
+  /** Closes the accordion panel with the specified value. */
   close(itemValue: string) {
-    this._pattern.expansionManager.close(this._patternForValue(itemValue)!);
+    const trigger = this._findTriggerPatternByValue(itemValue);
+
+    if (trigger) {
+      this._pattern.expansionManager.close(trigger);
+    }
   }
 
-  /** Toggles the expansion state of the specified item. */
+  /** Toggles the expansion state of the accordion panel with the specified value. */
   toggle(itemValue: string) {
-    this._pattern.expansionManager.toggle(this._patternForValue(itemValue)!);
+    const trigger = this._findTriggerPatternByValue(itemValue);
+
+    if (trigger) {
+      this._pattern.expansionManager.toggle(trigger);
+    }
   }
 
-  /** Opens all focusable items in the list. */
+  /** Opens all accordion panels if multi-expandable. */
   openAll() {
     this._pattern.expansionManager.openAll();
   }
 
-  /** Closes all focusable items in the list. */
+  /** Closes all accordion panels. */
   closeAll() {
     this._pattern.expansionManager.closeAll();
   }
 
-  _patternForValue(value: string) {
+  _findTriggerPatternByValue(value: string) {
     const trigger = this._triggers().find(t => t.value() === value);
+
     return trigger?._pattern;
   }
 }