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

test: indicate progress to screen reader users #210661

Closed
3 tasks done
meganrogge opened this issue Apr 18, 2024 · 2 comments
Closed
3 tasks done

test: indicate progress to screen reader users #210661

meganrogge opened this issue Apr 18, 2024 · 2 comments

Comments

@meganrogge
Copy link
Contributor

meganrogge commented Apr 18, 2024

Refs #209661, #196396, #210590

Complexity: 4

Create Issue


Formerly, progress information wasn't communicated to screen reader users. Now, anywhere we have a progress bar, if it takes more than seconds, a progress signal will be played and then repeat every five seconds.

This happens when something is very slow, such as:

  • the first time a notebook editor is loaded
  • running a notebook cell with a sleep call or a lot of processing
  • running an @workspace chat request

The sounds will play in parallel if several things are in progress simultaneously.

  • Enable a screen reader. Verify that any time a progress bar is shown for more than 3 seconds, you hear this sound.
  • Disable this accessibility.signals.progress setting. Verify that you don't hear it.
  • Disable screen reader mode. Verify that you don't hear it.
  • Add this to your settings file, reload the window, verify that the migration to accessibility.signals.progress has succeeeded
accessibility.signals.chatResponsePending: {
   "sound": "off",
   "announcement": "off"
}
@meganrogge meganrogge added this to the May 2024 milestone Apr 18, 2024
@dbaeumer dbaeumer removed their assignment Apr 23, 2024
@eleanorjboyd
Copy link
Member

When I try and put " "accessibility.signals.chatResponsePending": {
"sound": "off",
"announcement": "off"
}" in my settings (both workspace and user level) it grays out and says "Unknown Configuration Setting". Is there something else I need to enable?

@roblourens
Copy link
Member

That seems by design @eleanorjboyd, it's deprecated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants