fix(overflow-menu): add aria-controls to trigger button (#2100)

This commit is contained in:
Eric Liu 2025-02-17 13:40:42 -08:00 committed by GitHub
commit b7297d452a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -162,6 +162,7 @@
onMountAfterUpdate = false;
});
$: menuId = `menu-${id}`;
$: ariaLabel = $$props["aria-label"] || "menu";
$: if ($items[$currentIndex]) {
focusedId.set($items[$currentIndex].id);
@ -194,6 +195,7 @@
aria-haspopup="true"
aria-expanded={open}
aria-label={ariaLabel}
aria-controls={open ? menuId : undefined}
{id}
class:bx--overflow-menu={true}
class:bx--overflow-menu--open={open}
@ -239,6 +241,7 @@
bind:this={menuRef}
role="menu"
tabindex="-1"
id={menuId}
aria-label={ariaLabel}
data-floating-menu-direction={direction}
class:bx--overflow-menu-options={true}