@fluentui/web-components 3.0.0-beta.33 → 3.0.0-beta.34

package/CHANGELOG.md

@@ -1,12 +1,21 @@
 # Change Log - @fluentui/web-components
 
-This log was last generated on Fri, 21 Jun 2024 04:06:50 GMT and should not be manually modified.
+This log was last generated on Mon, 24 Jun 2024 04:05:55 GMT and should not be manually modified.
 
 <!-- Start content -->
 
+## [3.0.0-beta.34](https://github.com/microsoft/fluentui/tree/@fluentui/web-components_v3.0.0-beta.34)
+
+Mon, 24 Jun 2024 04:05:55 GMT 
+[Compare changes](https://github.com/microsoft/fluentui/compare/@fluentui/web-components_v3.0.0-beta.33..@fluentui/web-components_v3.0.0-beta.34)
+
+### Changes
+
+- Refactor Menu, MenuList, and MenuItem components ([PR #31674](https://github.com/microsoft/fluentui/pull/31674) by rupertdavid@microsoft.com)
+
 ## [3.0.0-beta.33](https://github.com/microsoft/fluentui/tree/@fluentui/web-components_v3.0.0-beta.33)
 
-Fri, 21 Jun 2024 04:06:50 GMT 
+Fri, 21 Jun 2024 04:06:58 GMT 
 [Compare changes](https://github.com/microsoft/fluentui/compare/@fluentui/web-components_v3.0.0-beta.32..@fluentui/web-components_v3.0.0-beta.33)
 
 ### Changes

package/dist/dts/menu/menu.d.ts

@@ -1,7 +1,43 @@
+/// <reference types="web" />
 import { FASTElement } from '@microsoft/fast-element';
 import { MenuList } from '../menu-list/menu-list.js';
 /**
- * The Menu class represents a menu component.
+ * A Menu component that provides a customizable menu element.
+ * @class Menu
+ * @extends FASTElement
+ *
+ * @attr open-on-hover - Determines if the menu should open on hover.
+ * @attr open-on-context - Determines if the menu should open on right click.
+ * @attr close-on-scroll - Determines if the menu should close on scroll.
+ * @attr persist-on-item-click - Determines if the menu open state should persist on click of menu item.
+ *
+ * @cssproperty --menu-max-height - The max-height of the menu.
+ *
+ * @slot trigger - Slot for the trigger elements.
+ * @slot - Default slot for the menu list.
+ *
+ * @method connectedCallback - Called when the element is connected to the DOM. Sets up the component.
+ * @method disconnectedCallback - Called when the element is disconnected from the DOM. Removes event listeners.
+ * @method setComponent - Sets the component state.
+ * @method toggleMenu - Toggles the open state of the menu.
+ * @method closeMenu - Closes the menu.
+ * @method openMenu - Opens the menu.
+ * @method focusMenuList - Focuses on the menu list.
+ * @method focusTrigger - Focuses on the menu trigger.
+ * @method openOnHoverChanged - Called whenever the 'openOnHover' property changes.
+ * @method persistOnItemClickChanged - Called whenever the 'persistOnItemClick' property changes.
+ * @method openOnContextChanged - Called whenever the 'openOnContext' property changes.
+ * @method closeOnScrollChanged - Called whenever the 'closeOnScroll' property changes.
+ * @method addListeners - Adds event listeners.
+ * @method removeListeners - Removes event listeners.
+ * @method menuKeydownHandler - Handles keyboard interaction for the menu.
+ * @method triggerKeydownHandler - Handles keyboard interaction for the trigger.
+ * @method documentClickHandler - Handles document click events to close the menu when a click occurs outside of the menu or the trigger.
+ *
+ * @summary The Menu component functions as a customizable menu element.
+ *
+ * @tag fluent-menu
+ *
  * @public
  */
 export declare class Menu extends FASTElement {
@@ -25,11 +61,6 @@ export declare class Menu extends FASTElement {
      * @public
      */
     persistOnItemClick?: boolean;
-    /**
-     * Defines whether the menu is open or not.
-     * @public
-     */
-    open: boolean;
     /**
      * Holds the slotted menu list.
      * @public
@@ -41,25 +72,20 @@ export declare class Menu extends FASTElement {
      */
     slottedTriggers: HTMLElement[];
     /**
-     * The positioning container of the menu.
+     * Defines whether the menu is open or not.
      * @internal
      */
-    positioningContainer?: HTMLElement;
+    private _open;
     /**
      * The trigger element of the menu.
      * @internal
      */
     private _trigger?;
     /**
-     * The menu list element of the menu.
+     * The menu list element of the menu which has the popover behavior.
      * @internal
      */
     private _menuList?;
-    /**
-     * Holds a reference to a function that is used to cleanup resources.
-     * @public
-     */
-    cleanup?: () => void;
     /**
      * Called when the element is connected to the DOM.
      * Sets up the component.
@@ -104,16 +130,12 @@ export declare class Menu extends FASTElement {
      */
     focusTrigger(): void;
     /**
-     * Called whenever the open state changes.
-     * Updates the 'aria-expanded' attribute and sets the positioning of the menu.
-     * Sets menu list position
-     * emits openChanged event
-     *
-     * @param oldValue - The previous value of 'open'.
-     * @param newValue - The new value of 'open'.
+     * Handles the 'toggle' event on the popover.
      * @public
+     * @param e - the event
+     * @returns void
      */
-    openChanged(oldValue: boolean, newValue: boolean): void;
+    toggleHandler: (e: Event | ToggleEvent) => void;
     /**
      * Called whenever the 'openOnHover' property changes.
      * Adds or removes a 'mouseover' event listener to the trigger based on the new value.
@@ -147,27 +169,21 @@ export declare class Menu extends FASTElement {
      * @param newValue - The new value of 'closeOnScroll'.
      */
     closeOnScrollChanged(oldValue: boolean, newValue: boolean): void;
-    /**
-     * The task to set the positioning of the menu.
-     * @internal
-     */
-    protected setPositioningTask: () => void;
-    /**
-     * Sets the positioning of the menu.
-     * @internal
-     */
-    protected setPositioning(): void;
     /**
      * Adds event listeners.
-     * Adds click and keydown event listeners to the trigger and a click event listener to the document.
+     * Adds click and keydown event listeners to the trigger.
+     * Adds a 'toggle' event listener to the menu list.
      * If 'openOnHover' is true, adds a 'mouseover' event listener to the trigger.
-     * @public
+     * If 'openOnContext' is true, adds a 'contextmenu' event listener to the trigger and a document 'click' event listener.
+     * @internal
      */
     private addListeners;
     /**
      * Removes event listeners.
-     * Removes click and keydown event listeners from the trigger and a click event listener from the document.
+     * Removes click and keydown event listeners from the trigger.
+     * Also removes toggle event listener from the menu list.
      * Also removes 'mouseover' event listeners from the trigger.
+     * Also removes 'contextmenu' event listeners from the trigger and document 'click' event listeners.
      * @internal
      */
     private removeListeners;
@@ -178,7 +194,7 @@ export declare class Menu extends FASTElement {
      * @param e - the keyboard event
      * @public
      */
-    handleMenuKeydown(e: KeyboardEvent): boolean | void;
+    menuKeydownHandler(e: KeyboardEvent): boolean | void;
     /**
      * Handles keyboard interaction for the trigger. Toggles the menu when the Space or Enter key is pressed. If the menu
      * is open, focuses on the menu list.
@@ -186,11 +202,11 @@ export declare class Menu extends FASTElement {
      * @param e - the keyboard event
      * @public
      */
-    handleTriggerKeydown: (e: KeyboardEvent) => boolean | void;
+    triggerKeydownHandler: (e: KeyboardEvent) => boolean | void;
     /**
-     * Handles document click events to close the menu when a click occurs outside of the menu or the trigger.
+     * Handles document click events to close a menu opened with contextmenu in popover="manual" mode.
      * @internal
      * @param e - The event triggered on document click.
      */
-    private handleDocumentClick;
+    private documentClickHandler;
 }

package/dist/dts/menu-item/menu-item.d.ts

@@ -1,3 +1,4 @@
+/// <reference types="web" />
 import { FASTElement } from '@microsoft/fast-element';
 import type { StaticallyComposableHTML } from '../utils/template-helpers.js';
 import type { StartEndOptions } from '../patterns/start-end.js';
@@ -10,29 +11,20 @@ export { MenuItemRole, roleForMenuItem };
  * @public
  */
 export type MenuItemOptions = StartEndOptions<MenuItem> & {
-    checkboxIndicator?: StaticallyComposableHTML<MenuItem>;
-    expandCollapseGlyph?: StaticallyComposableHTML<MenuItem>;
-    radioIndicator?: StaticallyComposableHTML<MenuItem>;
+    indicator?: StaticallyComposableHTML<MenuItem>;
+    submenuGlyph?: StaticallyComposableHTML<MenuItem>;
 };
 /**
  * A Switch Custom HTML Element.
  * Implements {@link https://www.w3.org/TR/wai-aria-1.1/#menuitem | ARIA menuitem }, {@link https://www.w3.org/TR/wai-aria-1.1/#menuitemcheckbox | ARIA menuitemcheckbox}, or {@link https://www.w3.org/TR/wai-aria-1.1/#menuitemradio | ARIA menuitemradio }.
  *
- * @slot checked-indicator - The checked indicator
- * @slot radio-indicator - The radio indicator
+ * @slot indicator - The checkbox or radio indicator
  * @slot start - Content which can be provided before the menu item content
- * @slot end - Content which can be provided after the menu item content
  * @slot - The default slot for menu item content
- * @slot expand-collapse-indicator - The expand/collapse indicator
+ * @slot end - Content which can be provided after the menu item content
+ * @slot submenu-glyph - The submenu expand/collapse indicator
  * @slot submenu - Used to nest menu's within menu items
- * @csspart input-container - The element representing the visual checked or radio indicator
- * @csspart checkbox - The element wrapping the `menuitemcheckbox` indicator
- * @csspart radio - The element wrapping the `menuitemradio` indicator
  * @csspart content - The element wrapping the menu item content
- * @csspart expand-collapse-glyph-container - The element wrapping the expand collapse element
- * @csspart expand-collapse - The expand/collapse element
- * @csspart submenu-region - The container for the submenu, used for positioning
- * @fires expanded-change - Fires a custom 'expanded-change' event when the expanded state changes
  * @fires change - Fires a custom 'change' event when a non-submenu item with a role of `menuitemcheckbox`, `menuitemradio`, or `menuitem` is invoked
  *
  * @public
@@ -46,15 +38,6 @@ export declare class MenuItem extends FASTElement {
      * HTML Attribute: disabled
      */
     disabled: boolean;
-    /**
-     * The expanded state of the element.
-     *
-     * @public
-     * @remarks
-     * HTML Attribute: expanded
-     */
-    expanded: boolean;
-    protected expandedChanged(prev: boolean | undefined, next: boolean): void;
     /**
      * The role of the element.
      *
@@ -63,12 +46,6 @@ export declare class MenuItem extends FASTElement {
      * HTML Attribute: role
      */
     role: MenuItemRole;
-    /**
-     * Cleanup function for the submenu positioner.
-     *
-     * @public
-     */
-    cleanup: () => void;
     /**
      * The checked value of the element.
      *
@@ -92,31 +69,16 @@ export declare class MenuItem extends FASTElement {
      * @internal
      */
     slottedSubmenu: HTMLElement[];
-    /**
-     * @internal
-     */
-    get hasSubmenu(): boolean;
     /**
      * Sets the submenu and updates its position.
      *
      * @internal
      */
     protected slottedSubmenuChanged(prev: HTMLElement[] | undefined, next: HTMLElement[]): void;
-    /**
-     * The container for the submenu.
-     *
-     * @internal
-     */
-    submenuContainer: HTMLDivElement;
     /**
      * @internal
      */
     submenu: HTMLElement | undefined;
-    private focusSubmenuOnLoad;
-    /**
-     * @internal
-     */
-    disconnectedCallback(): void;
     /**
      * @internal
      */
@@ -125,10 +87,6 @@ export declare class MenuItem extends FASTElement {
      * @internal
      */
     handleMenuItemClick: (e: MouseEvent) => boolean;
-    /**
-     * @internal
-     */
-    submenuLoaded: () => void;
     /**
      * @internal
      */
@@ -138,23 +96,19 @@ export declare class MenuItem extends FASTElement {
      */
     handleMouseOut: (e: MouseEvent) => boolean;
     /**
+     * Setup required ARIA on open/close
      * @internal
      */
-    private closeSubMenu;
-    /**
-     * @internal
-     */
-    private expandAndFocus;
+    toggleHandler: (e: ToggleEvent | Event) => void;
     /**
      * @internal
      */
     private invoke;
     /**
-     * Calculate and apply submenu positioning.
-     *
-     * @public
+     * Set fallback position of menu on open when CSS anchor not supported
+     * @internal
      */
-    updateSubmenu(): void;
+    setSubmenuPosition: () => void;
 }
 /**
  * Mark internal because exporting class and interface of the same name

package/dist/dts/menu-list/menu-list.d.ts

@@ -14,7 +14,6 @@ export declare class MenuList extends FASTElement {
     items: HTMLElement[];
     protected itemsChanged(oldValue: HTMLElement[], newValue: HTMLElement[]): void;
     protected menuItems: Element[] | undefined;
-    private expandedItem;
     /**
      * The index of the focusable element in the items array
      * defaults to -1
@@ -39,12 +38,6 @@ export declare class MenuList extends FASTElement {
      * @public
      */
     focus(): void;
-    /**
-     * Collapses any expanded menu items.
-     *
-     * @public
-     */
-    collapseExpandedItem(): void;
     /**
      * @internal
      */
@@ -55,15 +48,17 @@ export declare class MenuList extends FASTElement {
      */
     handleFocusOut: (e: FocusEvent) => void;
     private handleItemFocus;
-    private handleExpandedChanged;
     private removeItemListeners;
     private static elementIndent;
     protected setItems(): void;
+    /**
+     * Method for Observable changes to the hidden attribute of child elements
+     */
     handleChange(source: any, propertyName: string): void;
     /**
-     * handle change from child element
+     * Handle change from child MenuItem element and set radio group behavior
      */
-    private changeHandler;
+    private changedMenuItemHandler;
     /**
      * check if the item is a menu item
      */