Skip to content
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

bug(mat-progress-bar): ChromeVox can't read a progress bar #22165

Closed
jamOne- opened this issue Mar 9, 2021 · 1 comment · Fixed by #22166
Closed

bug(mat-progress-bar): ChromeVox can't read a progress bar #22165

jamOne- opened this issue Mar 9, 2021 · 1 comment · Fixed by #22166
Labels
Accessibility This issue is related to accessibility (a11y) area: material/progress-bar G This is is related to a Google internal issue P2 The issue is important to a large percentage of users, with a workaround

Comments

@jamOne-
Copy link
Contributor

jamOne- commented Mar 9, 2021

Reproduction

Steps to reproduce:

  1. Go to link https://material.angular.io/components/progress-bar/overview#determinate.
  2. Enable ChromeVox.
  3. Try to focus the progress bar with the screen reader.

Expected Behavior

ChromeVox should announce progress bar element with its state.

Actual Behavior

ChromeVox skips this element while navigating through a page.

Environment

  • Browser(s): Chrome 88.0.4324.186 (Official Build) (64-bit)
  • Operating System (e.g. Windows, macOS, Ubuntu): Chrome OS

Additional note

When I add aria-hidden="true" attribute to every child of mat-progress-bar element the problem disappears.
I tried filing a bug against Chrome OS team link, but haven't received any response since Jun, so decided to try the other way around, with making a change to angular/components.

@jamOne- jamOne- added the needs triage This issue needs to be triaged by the team label Mar 9, 2021
@jelbourn jelbourn added Accessibility This issue is related to accessibility (a11y) area: material/progress-bar G This is is related to a Google internal issue P2 The issue is important to a large percentage of users, with a workaround has pr and removed needs triage This issue needs to be triaged by the team labels Mar 9, 2021
andrewseguin pushed a commit that referenced this issue Mar 19, 2021
* fix(material/progress-bar): Support ChromeVox

* refactor(material/progress-bar): wrap elements with aria-hidden div

* fix(material-experimental/mdc-progress-bar): Support ChromeVox #22165

* test(material/progress-bar): add ChromeVox support tests

* test(material-experimental/mdc-progress-bar): add ChromeVox support test
andrewseguin pushed a commit that referenced this issue Mar 19, 2021
* fix(material/progress-bar): Support ChromeVox

* refactor(material/progress-bar): wrap elements with aria-hidden div

* fix(material-experimental/mdc-progress-bar): Support ChromeVox #22165

* test(material/progress-bar): add ChromeVox support tests

* test(material-experimental/mdc-progress-bar): add ChromeVox support test

(cherry picked from commit 5b7b03e)
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Accessibility This issue is related to accessibility (a11y) area: material/progress-bar G This is is related to a Google internal issue P2 The issue is important to a large percentage of users, with a workaround
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants