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

Review scrollable listbox example #469

Closed
3 tasks done
mcking65 opened this issue Oct 6, 2017 · 4 comments
Closed
3 tasks done

Review scrollable listbox example #469

mcking65 opened this issue Oct 6, 2017 · 4 comments
Assignees
Labels
Example Page Related to a page containing an example implementation of a pattern

Comments

@mcking65
Copy link
Contributor

mcking65 commented Oct 6, 2017

The scrollable listbox example
developed for issue #123 is ready for task force review.

Reviews Requested as of October 6, 2017

While the practices task force encourages feedback from anyone in the web and accessibility engineering communities, peer review is requested from the following task force members:

@mcking65 mcking65 added Needs Review Example Page Related to a page containing an example implementation of a pattern labels Oct 6, 2017
@mcking65 mcking65 added this to the 3Q17 Working Draft milestone Oct 6, 2017
@annabbott
Copy link

In the Keyboard Support table, only "Down Arrow" has a bullet. Suggest removing that bullet.

mcking65 added a commit that referenced this issue Oct 10, 2017
…ard Table

Per feedback from @annabbott in issue #469, modified examples/listbox/listbox-scrollable.html
to remove the list markup from the down arrow row in the keyboard table.
@shirsha
Copy link

shirsha commented Oct 12, 2017

Tested in FF browser with NVDA screen reader:

  • In the read only mode when screen reader encounter the listbox, screen reader announces the box as list instead of listbox even though there is role of listbox.
  • As there is tabindex of "0" on the box, screen reader announces it as clickable, when the user selects enter, he/she can navigate through the list using up / down arrow. But user is not aware that they can make a selection here, it reads as a normal list items in an unordered list.
    Suggest to add aria-describedby to convey the information regarding the ability to select an item in the list. This will help the screen reader user if he/she navigates directly to the box by tabbing to it .

Tested in IE using JAWS:

  • JAWS announces it as listbox but screen reader user is unable to navigate through the list. The only way the user can navigate through the list in the read only mode it to select tab (instead of selecting enter to switch to forms mode) and then use up/down arrow .
    JAWS does not read that it is clickable when it encounters the box.

@mcking65
Copy link
Contributor Author

Siri, thank you for the review and testing!

@shirsha commented:

Tested in FF browser with NVDA screen reader:

In the read only mode when screen reader encounter the listbox, screen reader announces the box as list instead of listbox even though there is role of listbox.
As there is tabindex of "0" on the box, screen reader announces it as clickable, when the user selects enter, he/she can navigate through the list using up / down arrow. But user is not aware that they can make a selection here, it reads as a normal list items in an unordered list.

This is an NVDA design decision. They do not have to use the exact name of the role. They provide information to their users on how to use list widgets. And, if they like, they can also provide dynamic instructions. Web page authors should not provide instructions unless the design does something outside normal patterns that the screen reader cannot derive from the semantics provided by the ARIA.

Tested in IE using JAWS:

  • JAWS announces it as listbox but screen reader user is unable to navigate through the list. The only way the user can navigate through the list in the read only mode it to select tab (instead of selecting enter to switch to forms mode) and then use up/down arrow .
  • JAWS does not read that it is clickable when it encounters the box.

This is definitely a JAWS bug ... it is pretty bad because JAWS does not set focus correctly when pressing enter when the vc is active. In other browsers, it would be a browser bug, but in IE, JAWS doesn't rely on MSAA for these functions.

NVDA has a pretty bad IE bug here too as it does not recognize aria-selected of true in the listbox; it reads the selected item as not selected.

mcking65 added a commit that referenced this issue Oct 30, 2017
Modified examples/listbox/listbox-scrollable.html
to remove link to review issue #469.
@mcking65
Copy link
Contributor Author

Task force review of this example is now complete. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Example Page Related to a page containing an example implementation of a pattern
Development

No branches or pull requests

4 participants