Skip to content

fix(aria/menu): defer menu item focus in case menus in cdk overlay#33258

Merged
ok7sai merged 1 commit into
angular:mainfrom
ok7sai:fix/aria-menu-overlay
May 18, 2026
Merged

fix(aria/menu): defer menu item focus in case menus in cdk overlay#33258
ok7sai merged 1 commit into
angular:mainfrom
ok7sai:fix/aria-menu-overlay

Conversation

@ok7sai
Copy link
Copy Markdown
Member

@ok7sai ok7sai commented May 14, 2026

When menus are placed in a CdkConnectedOverlay, the menu trigger tries to focus the first menu item even before the menus are rendered and registered to the menu trigger.

@ok7sai ok7sai force-pushed the fix/aria-menu-overlay branch 2 times, most recently from 6a24e03 to 7d0ba63 Compare May 14, 2026 23:41
@ok7sai ok7sai marked this pull request as ready for review May 14, 2026 23:54
@pullapprove pullapprove Bot requested review from andrewseguin and tjshiu May 14, 2026 23:55
Comment thread src/aria/private/menu/menu.spec.ts
@ok7sai ok7sai added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate labels May 18, 2026
When menus are placed in a `CdkConnectedOverlay`, the menu trigger tries
to focus the first menu item even before the menus are rendered and
registered to the menu trigger.
@ok7sai ok7sai force-pushed the fix/aria-menu-overlay branch from 7d0ba63 to da5b97b Compare May 18, 2026 05:34
@ok7sai ok7sai merged commit 629fe71 into angular:main May 18, 2026
22 of 24 checks passed
@ok7sai
Copy link
Copy Markdown
Member Author

ok7sai commented May 18, 2026

This PR was merged into the repository. The changes were merged into the following branches:

ok7sai added a commit that referenced this pull request May 18, 2026
…33258)

When menus are placed in a `CdkConnectedOverlay`, the menu trigger tries
to focus the first menu item even before the menus are rendered and
registered to the menu trigger.

(cherry picked from commit 629fe71)
@ok7sai ok7sai deleted the fix/aria-menu-overlay branch May 18, 2026 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: aria/menu target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants