Skip to content

fix(material/chips): provide ability to edit for all screen readers with a click on already focused chip #30983

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.

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

Merged
merged 1 commit into from
May 29, 2025

Conversation

adolgachev
Copy link
Contributor

@adolgachev adolgachev commented Apr 29, 2025

Some screen readers have difficulty with allowing users to say double click or enter with voice control, preventing them from being able to edit a chip which currently requires a double click or an enter key event.

Single click conflicts with other behaviors, but allowing a click when the chips has already been focused is a natural way to edit the chip and does not affect other current behaviors.

Fixes b/290806246.

@adolgachev adolgachev added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label Apr 29, 2025
Copy link

github-actions bot commented Apr 29, 2025

Deployed dev-app for 291d8fe to: https://ng-dev-previews-comp--pr-angular-components-30983-dev-9kax7hje.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@adolgachev adolgachev changed the title Begin editing on click on focused chip fix(chips): provide ability to edit for all screen readers with a click on already focused chip Apr 29, 2025
@adolgachev adolgachev changed the title fix(chips): provide ability to edit for all screen readers with a click on already focused chip fix(material/chips): provide ability to edit for all screen readers with a click on already focused chip Apr 29, 2025
@adolgachev adolgachev added the requires: TGP This PR requires a passing TGP before merging is allowed label Apr 29, 2025
@adolgachev adolgachev marked this pull request as ready for review May 5, 2025 19:35
@adolgachev adolgachev requested a review from a team as a code owner May 5, 2025 19:35
@adolgachev adolgachev requested review from mmalerba, andrewseguin and crisbeto and removed request for a team May 5, 2025 19:35
@adolgachev adolgachev added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 6, 2025
@adolgachev adolgachev added the Accessibility This issue is related to accessibility (a11y) label May 7, 2025
@adolgachev adolgachev added action: global presubmit The PR is in need of a google3 global presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 8, 2025
@adolgachev adolgachev added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release and removed action: global presubmit The PR is in need of a google3 global presubmit labels May 29, 2025
@adolgachev adolgachev 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 May 29, 2025
@adolgachev adolgachev merged commit 68d7ea0 into angular:main May 29, 2025
29 of 31 checks passed
@adolgachev adolgachev deleted the chip-edit branch May 29, 2025 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker area: material/chips dev-app preview When applied, previews of the dev-app are deployed to Firebase requires: TGP This PR requires a passing TGP before merging is allowed 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