Bug 491209 - SideMenu needs navigation role and not sideMenuList
Summary: SideMenu needs navigation role and not sideMenuList
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: ---   Edit
Assignee: Carolyn MacLeod CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2016-04-07 03:58 EDT by Carolyn MacLeod CLA
Modified: 2016-05-17 14:47 EDT (History)
0 users

See Also:

Attachments
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 2016-04-07 03:58:07 EDT 
The sideMenu <div class="sideMenu" id="sideMenu"> 
needs to have navigation role (and aria-label) instead of the sideMenuList
<ul class="sideMenuList" role="navigation" aria-label="Navigation Bar">
because <ul> is not allowed to have navigation role.
Comment 1 Carolyn MacLeod CLA 2016-04-07 03:59:04 EDT 
sideMenuList is created in SideMenu.js line 116.
Comment 2 Carolyn MacLeod CLA 2016-05-05 22:37:49 EDT 
The whole sideMenu should follow the Tabbed Style Site Navigator pattern Option 1: Listbox containing links:
https://orion.eclipse.org/file/aria/practices/aria-practices.html#Site_Navigator_Tabbed_Style

Unfortunately, the button at the top (sideMenuHome) can't go into the list because of the scrolling. But the nav should contain sideMenuHome.