fixup! feat(aria): add initial menu directives

Author: wagnermaciel

src/aria/menu/menu.ts

@@ -57,6 +57,8 @@ export class MenuTrigger<V> {
   /** A reference to the menu element. */
   readonly element: HTMLButtonElement = this._elementRef.nativeElement;
 
+  // TODO(wagnermaciel): See we can remove the need to pass in a submenu.
+
   /** The submenu associated with the menu trigger. */
   submenu = input<Menu<V> | undefined>(undefined);
 
@@ -135,9 +137,6 @@ export class Menu<V> {
   /** Whether the menu should wrap its items. */
   readonly wrap = input<boolean>(true);
 
-  /** Whether the menu should skip disabled items. */
-  readonly skipDisabled = input<boolean>(false);
-
   /** The delay in seconds before the typeahead buffer is cleared. */
   readonly typeaheadDelay = input<number>(0.5); // Picked arbitrarily.
 
@@ -167,6 +166,7 @@ export class Menu<V> {
       ...this,
       parent: computed(() => this.parent()?.uiPattern),
       multi: () => false,
+      skipDisabled: () => false,
       focusMode: () => 'roving',
       orientation: () => 'vertical',
       selectionMode: () => 'explicit',
@@ -261,9 +261,6 @@ export class MenuBar<V> {
   /** Whether the menu should wrap its items. */
   readonly wrap = input<boolean>(true);
 
-  /** Whether the menu should skip disabled items. */
-  readonly skipDisabled = input<boolean>(false);
-
   /** The delay in seconds before the typeahead buffer is cleared. */
   readonly typeaheadDelay = input<number>(0.5);
 
@@ -280,6 +277,7 @@ export class MenuBar<V> {
     this.uiPattern = new MenuBarPattern({
       ...this,
       multi: () => false,
+      skipDisabled: () => false,
       focusMode: () => 'roving',
       orientation: () => 'horizontal',
       selectionMode: () => 'explicit',