refactor(aria/menu): Extend public api with open/close methods

Author: adolgachev

src/aria/menu/menu.ts

@@ -85,6 +85,16 @@ export class MenuTrigger<V> {
   onFocusIn() {
     this.hasBeenFocused.set(true);
   }
+
+  /** Opens the menu. */
+  open(opts?: {first?: boolean; last?: boolean}) {
+    this._pattern.open(opts);
+  }
+
+  /** Closes the menu. */
+  close(opts: {refocus?: boolean} = {}) {
+    this._pattern.close(opts);
+  }
 }
 
 /**
@@ -222,24 +232,14 @@ export class Menu<V> {
     });
   }
 
-  // TODO(wagnermaciel): Author close, closeAll, and open methods for each directive.
-
   /** Closes the menu. */
   close(opts?: {refocus?: boolean}) {
-    this._pattern.inputs.parent()?.close(opts);
+    this._pattern.close(opts);
   }
 
   /** Closes all parent menus. */
   closeAll(opts?: {refocus?: boolean}) {
-    const root = this._pattern.root();
-
-    if (root instanceof MenuTriggerPattern) {
-      root.close(opts);
-    }
-
-    if (root instanceof MenuPattern || root instanceof MenuBarPattern) {
-      root.inputs.activeItem()?.close(opts);
-    }
+    this._pattern.closeAll(opts);
   }
 }
 
@@ -320,6 +320,11 @@ export class MenuBar<V> {
       }
     });
   }
+
+  /** Closes the menubar and refocuses the root menu bar item. */
+  close(opts?: {refocus?: boolean}) {
+    this._pattern.close(opts);
+  }
 }
 
 /**
@@ -397,6 +402,16 @@ export class MenuItem<V> {
   onFocusIn() {
     this.hasBeenFocused.set(true);
   }
+
+  /** Opens the submenu. */
+  open(opts?: {first?: boolean; last?: boolean}) {
+    this._pattern.open(opts);
+  }
+
+  /** Closes the submenu. */
+  close(opts: {refocus?: boolean} = {}) {
+    this._pattern.close(opts);
+  }
 }
 
 /** Defers the rendering of the menu content. */

src/aria/private/menu/menu.ts

@@ -154,7 +154,7 @@ export class MenuPattern<V> {
       .on('Home', () => this.first())
       .on('End', () => this.last())
       .on('Enter', () => this.trigger())
-      .on('Escape', () => this.closeAll())
+      .on('Escape', () => this.closeAll({refocus: true}))
       .on(this._expandKey, () => this.expand())
       .on(this._collapseKey, () => this.collapse())
       .on(this.dynamicSpaceKey, () => this.trigger())
@@ -395,20 +395,25 @@ export class MenuPattern<V> {
     }
   }
 
+  /** Closes the menu. */
+  close(opts?: {refocus?: boolean}) {
+    this.inputs.parent()?.close(opts);
+  }
+
   /** Closes the menu and all parent menus. */
-  closeAll() {
+  closeAll(opts?: {refocus?: boolean}) {
     const root = this.root();
 
     if (root instanceof MenuTriggerPattern) {
-      root.close({refocus: true});
+      root.close(opts);
     }
 
     if (root instanceof MenuBarPattern) {
       root.close();
     }
 
     if (root instanceof MenuPattern) {
-      root.inputs.activeItem()?.close({refocus: true});
+      root.inputs.activeItem()?.close(opts);
     }
   }
 
@@ -568,8 +573,10 @@ export class MenuBarPattern<V> {
   }
 
   /** Closes the menubar and refocuses the root menu bar item. */
-  close() {
-    this.inputs.activeItem()?.close({refocus: this.isFocused()});
+  close(opts?: {refocus?: boolean}) {
+    opts ??= {refocus: this.isFocused()};
+
+    this.inputs.activeItem()?.close(opts);
   }
 }