Skip to content

fix(multiple): account for shadow dom in aria directives - #33691

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:aria-shadow-dom-target-again
Aug 17, 2026
Merged

fix(multiple): account for shadow dom in aria directives#33691
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:aria-shadow-dom-target-again

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Fixes that the Aria directives were reading target directly off the event which can break if the app is using shadow DOM.

Fixes that the Aria directives were reading `target` directly off the event which can break if the app is using shadow DOM.
@crisbeto
crisbeto requested a review from ok7sai August 17, 2026 13:57
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Aug 17, 2026
@pullapprove
pullapprove Bot requested a review from andrewseguin August 17, 2026 13:58
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Aug 17, 2026
@crisbeto
crisbeto merged commit 7dccecc into angular:main Aug 17, 2026
29 of 33 checks passed
@crisbeto

Copy link
Copy Markdown
Member Author

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

crisbeto added a commit that referenced this pull request Aug 17, 2026
Fixes that the Aria directives were reading `target` directly off the event which can break if the app is using shadow DOM.

(cherry picked from commit 7dccecc)
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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants