Skip to content

Commit c37162c

Browse files
Rajdeep ChandraRajdeep Chandra
authored andcommitted
fix: resetting esc key handler
1 parent 483d787 commit c37162c

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

packages/menu/src/MenuItem.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -583,9 +583,10 @@ export class MenuItem extends LikeAnchor(
583583
const openSubmenuKey =
584584
this.hasSubmenu && !this.open && [' ', 'Enter'].includes(key);
585585
if (target === this) {
586-
// Only prevent default for arrow keys and submenu-related keys.
587-
// Don't prevent Escape - let it bubble to close overlays.
588-
if (['ArrowLeft', 'ArrowRight'].includes(key) || openSubmenuKey)
586+
if (
587+
['ArrowLeft', 'ArrowRight', 'Escape'].includes(key) ||
588+
openSubmenuKey
589+
)
589590
event.preventDefault();
590591
this.dispatchEvent(
591592
new MenuItemKeydownEvent({ root: this, event: event })

0 commit comments

Comments
 (0)