Bug 491209

Summary: SideMenu needs navigation role and not sideMenuList
Product: [ECD] Orion (Archived) Reporter: Carolyn MacLeod <Carolyn_MacLeod>
Component: ClientAssignee: Carolyn MacLeod <Carolyn_MacLeod>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 Keywords: accessibility
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:

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.