Bug 475917 - Navigation bar: screen reader user can't tell which page is currently selected
Summary: Navigation bar: screen reader user can't tell which page is currently selected
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Client (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 15.0   Edit
Assignee: Carolyn MacLeod CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2015-08-26 08:35 EDT by Carolyn MacLeod CLA
Modified: 2017-06-07 14:34 EDT (History)
1 user (show)

See Also:

Attachments
screen snap: nav buttons need name.png (7.98 KB, image/png)
2015-08-26 08:35 EDT, Carolyn MacLeod CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2015-08-26 08:35:36 EDT 
Created attachment 256131 [details]
screen snap: nav buttons need name.png

When the user tabs through the navigation bar (Editor, Git Repo, Shell, Sites, Settings) the screen reader announces "link", and "visited" if the link has been visited in the past. This does not give any indication about which "page" is currently selected.

Setting aria-current="page" on the currently selected nav link should tell the screen reader that the focused link is the "current" link.

http://www.w3.org/TR/wai-aria-1.1/#aria-current
Comment 1 Carolyn MacLeod CLA 2015-08-26 08:45:45 EDT 
Except that aria-current isn't yet supported. Tried with FF & Chrome, JAWS & NVDA.
Comment 2 Carolyn MacLeod CLA 2015-08-26 08:49:47 EDT 
Probably the best way to solve this is to use aria-describedby
as described in this blog:
http://www.heydonworks.com/article/the-accessible-current-page-link-conundrum
Comment 3 Michael Rennie CLA 2017-01-10 15:45:36 EST 
Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see:

https://dev.eclipse.org/mhonarc/lists/orion-dev/msg04002.html
Comment 4 Carolyn MacLeod CLA 2017-01-14 18:03:51 EST 
Reopening, because aria-current now has screen reader support.
See the aria-current="page" example on this page:
https://ljwatson.github.io/design-patterns/aria-current/

For more info:
http://tink.uk/using-the-aria-current-attribute/
https://www.youtube.com/watch?v=f3tNnqSW3VM
Comment 5 Carolyn MacLeod CLA 2017-01-16 20:32:39 EST 
Support for aria-current is mostly there now: https://github.com/nvaccess/nvda/issues/6358#issuecomment-272981835
We should just add this, even if automated tooling doesn't recognize it yet.
Comment 6 Eclipse Genie CLA 2017-06-07 13:35:03 EDT 
GitHub Pull Request 49 created by [carmacleod]
https://github.com/eclipse/orion.client/pull/49
Comment 7 Carolyn MacLeod CLA 2017-06-07 14:34:59 EDT 
Fixed with PR in comment 6.