Skip to content

fix(material/checkbox): avoid making the touch target a focus stop - #33584

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:checkbox-touch-target
Jul 30, 2026
Merged

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

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Along the same lines as #33558.

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 checkbox so it doesn't have to rely on a click listener on the touch target by wrapping the input in the label instead.

Along the same lines as angular#33558.

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 checkbox 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 25, 2026
@pullapprove
pullapprove Bot requested review from andrewseguin and tjshiu July 25, 2026 07:21
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jul 30, 2026
@crisbeto
crisbeto merged commit 3fb737c into angular:main Jul 30, 2026
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 30, 2026
…33584)

Along the same lines as #33558.

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 checkbox 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 3fb737c)
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/checkbox 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