Skip to content

Tools Boards menu scroll updates for accessibility #9168

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 2 commits into from
Aug 26, 2019

Conversation

joew46167
Copy link

When a user uses the keyboard to navigate the menu, when you arrow up or down onto the scroll arrows, the scroll just starts scrolling the menu, scrolling a new board every 150 ms. The board names don’t get read by the screen reader as they scroll by. So the screen reader user has no idea what’s going on. They are just on a menu item that says “menu item” at the screen reader. If they hit the same arrow key again, they get to the Boards Manager… menu item, either by scrolling up or by scrolling down and wrapping around. The wrapping around isn’t an issue – the user at least now knows where they are again because it says more than “menu item”

A big issue is that if they have a small screen and a very large number of board installed, it is possible that the board that they are looking for would scroll onto the visible part of the menu and then back off the visible part as the scroll continues. After a bit of wondering what’s going on, because “screen item” made no sense, they may arrow back the other way to try to figure out what is going on. Going the other way they might notice that the boards had changed, and they would think that that is odd. And name of the board they want would never be read to them – it scrolled into and out of the visible area. They know their board is installed, but they just can’t get to it.

Ideally, the menu wouldn’t scroll continuously. Instead when they get to the end of the visible area and hit arrow again, only the next item would appear. And I could easily make that happed by just removing the scroll – then the screen reader just keeps going through the list, but they end up scrolling off of the screen. But for the low vision user that’s no good – they use both the screen reader and what they can see on the screen. Or if the person who is blind asks for sighted person’s help, the sighted person would want to see what’s going on.

So the problems isn’t that it goes back to the head of the list. The problem is that the scroll doesn’t tell the user that it’s scrolling. And as it scrolls it doesn’t tell the user WHAT is scrolling. So this is intended to scroll the list one item at a time AND read that item name. While still maintaining usability for low vision and sighted users.

I hope that made sense.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Does your submission pass tests?
  2. Have you lint your code locally prior to submission?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes

@joew46167
Copy link
Author

Test in windows with NVDA and JAWS

@facchinm facchinm merged commit 930ec16 into arduino:master Aug 26, 2019
@cmaglie cmaglie added this to the Release 1.8.11 milestone Jan 22, 2020
@per1234 per1234 added accessibility Enabling the use of the software by everyone Component: IDE user interface The Arduino IDE's user interface labels Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Enabling the use of the software by everyone Component: IDE user interface The Arduino IDE's user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants