Skip to content

fix(material/radio): avoid making the touch target a focus stop#33558

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:radio-touch-target-click
Jul 20, 2026
Merged

fix(material/radio): avoid making the touch target a focus stop#33558
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:radio-touch-target-click

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Some screen readers detect which elements have click listeners and make them a focus stop, even if they're aria-hidden. These changes rework the radio button so it doesn't have to rely on a click listener on the touch target by wrapping the input in the label instead.

Some screen readers detect which elements have `click` listeners and make them a focus stop, even if they're `aria-hidden`. These changes rework the radio button so it doesn't have to rely on a `click` listener on the touch target by wrapping the `input` in the `label` instead.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Jul 20, 2026
@pullapprove
pullapprove Bot requested review from andrewseguin and ok7sai July 20, 2026 15:41
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jul 20, 2026
@crisbeto
crisbeto merged commit a62e659 into angular:main Jul 20, 2026
33 of 36 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 Jul 20, 2026
Some screen readers detect which elements have `click` listeners and make them a focus stop, even if they're `aria-hidden`. These changes rework the radio button so it doesn't have to rely on a `click` listener on the touch target by wrapping the `input` in the `label` instead.

(cherry picked from commit a62e659)
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/radio 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