Skip to content

fix(aria/menu): focus flicker bug #32406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
wagnermaciel merged 2 commits into from
Dec 3, 2025
Merged

fix(aria/menu): focus flicker bug #32406

wagnermaciel merged 2 commits into from
Dec 3, 2025

Conversation

@wagnermaciel
Copy link
Contributor

@wagnermaciel wagnermaciel commented Nov 25, 2025

  • Fixes an issue in standalone menus and context menus where focus would flicker when opening a menu.

@wagnermaciel wagnermaciel requested a review from a team as a code owner November 25, 2025 21:54
@wagnermaciel wagnermaciel requested review from crisbeto, mmalerba and tjshiu and removed request for a team November 25, 2025 21:54
@wagnermaciel wagnermaciel added merge: preserve commits When the PR is merged, a rebase and merge should be performed target: patch This PR is targeted for the next patch release labels Nov 25, 2025
@wagnermaciel
fix(aria/menu): update unit tests to use ngMenuContent
7d126f7
@wagnermaciel
fix(aria/menu): focus flicker bug
8949a16 
* Fixes an issue in standalone menus and context menus where focus would flicker when opening a menu.
@wagnermaciel wagnermaciel requested a review from ok7sai December 1, 2025 17:16
@tjshiu tjshiu removed their request for review December 1, 2025 18:58
@wagnermaciel wagnermaciel added the action: merge The PR is ready for merge by the caretaker label Dec 3, 2025
@wagnermaciel wagnermaciel merged commit 8533f42 into angular:main Dec 3, 2025
24 of 26 checks passed
@wagnermaciel
Copy link
Contributor Author

wagnermaciel commented Dec 3, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ok7sai ok7sai ok7sai approved these changes

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker merge: preserve commits When the PR is merged, a rebase and merge should be performed target: patch This PR is targeted for the next patch release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wagnermaciel @ok7sai