Skip to content

Clean up modal workarounds#33062

Merged
crisbeto merged 2 commits intoangular:mainfrom
crisbeto:modal-workarounds
Apr 10, 2026
Merged

Clean up modal workarounds#33062
crisbeto merged 2 commits intoangular:mainfrom
crisbeto:modal-workarounds

Conversation

@crisbeto
Copy link
Copy Markdown
Member

@crisbeto crisbeto commented Apr 9, 2026

A few months ago we switched to rendering the select and autocomplete modals inline which means that we don't need the aria-owns workaround anymore.

@crisbeto crisbeto added target: patch This PR is targeted for the next patch release merge: preserve commits When the PR is merged, a rebase and merge should be performed labels Apr 9, 2026
@pullapprove pullapprove bot requested review from adolgachev and ok7sai April 9, 2026 12:30
@crisbeto crisbeto added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Apr 9, 2026
crisbeto added 2 commits April 9, 2026 18:51
Now that we're rendering the autocomplete popover inline, we no longer need the `aria-owns` workaround.
Now that we're rendering the select popover inline, we no longer need the `aria-owns` workaround.
@crisbeto crisbeto force-pushed the modal-workarounds branch from 995dc0a to 300f6c4 Compare April 9, 2026 16:52
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Apr 10, 2026
@crisbeto crisbeto merged commit 348c3c8 into angular:main Apr 10, 2026
24 of 28 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

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

Labels

action: merge The PR is ready for merge by the caretaker area: material/autocomplete area: material/select merge: preserve commits When the PR is merged, a rebase and merge should be performed target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants