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

Author: adolgachev

src/aria/menu/menu.ts

@@ -83,6 +83,21 @@ export class MenuTrigger<V> {
   constructor() {
     effect(() => this.menu()?.parent.set(this));
   }
+
+  /** Marks the menu trigger as having been focused. */
+  onFocusIn() {
+    this.hasBeenFocused.set(true);
+  }
+
+  /** Opens the menu focusing on the first menu item. */
+  open() {
+    this._pattern.open({first: true});
+  }
+
+  /** Closes the menu. */
+  close() {
+    this._pattern.close();
+  }
 }
 
 /**
@@ -220,24 +235,9 @@ 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);
-  }
-
-  /** 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);
-    }
+  close() {
+    this._pattern.close();
   }
 }
 
@@ -318,6 +318,11 @@ export class MenuBar<V> {
       }
     });
   }
+
+  /** Closes the menubar. */
+  close() {
+    this._pattern.close();
+  }
 }
 
 /**
@@ -396,6 +401,21 @@ export class MenuItem<V> {
   constructor() {
     effect(() => this.submenu()?.parent.set(this));
   }
+
+  /** Marks the menu item as having been focused. */
+  onFocusIn() {
+    this.hasBeenFocused.set(true);
+  }
+
+  /** Opens the submenu focusing on the first menu item. */
+  open() {
+    this._pattern.open({first: true});
+  }
+
+  /** Closes the submenu. */
+  close() {
+    this._pattern.close();
+  }
 }
 
 /** Defers the rendering of the menu content. */

src/aria/private/menu/menu.ts

@@ -395,6 +395,11 @@ export class MenuPattern<V> {
     }
   }
 
+  /** Closes the menu. */
+  close() {
+    this.inputs.parent()?.close();
+  }
+
   /** Closes the menu and all parent menus. */
   closeAll() {
     const root = this.root();

src/components-examples/aria/menu/menu-context/menu-context-example.ts

@@ -37,7 +37,7 @@ export class MenuContextExample {
 
   open(event: MouseEvent) {
     const menu = this.menu();
-    menu?.closeAll();
+    menu?._pattern.closeAll();
 
     if (menu) {
       event.preventDefault();