Skip to content

Commit

Permalink
For accordion menu, changes aria roles from "tab" to "menuitem" and t…
Browse files Browse the repository at this point in the history
…abpanel" and "tablist" to "menu."
  • Loading branch information
MichellanneLi authored and kball committed Oct 20, 2016
1 parent eef3a53 commit 163bb73
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions js/foundation.accordionMenu.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class AccordionMenu {
_init() {
this.$element.find('[data-submenu]').not('.is-active').slideUp(0);//.find('a').css('padding-left', '1rem');
this.$element.attr({
'role': 'tablist',
'role': 'menu',
'aria-multiselectable': this.options.multiOpen
});

Expand All @@ -61,13 +61,13 @@ class AccordionMenu {
$elem.attr({
'aria-controls': subId,
'aria-expanded': isActive,
'role': 'tab',
'role': 'menuitem',
'id': linkId
});
$sub.attr({
'aria-labelledby': linkId,
'aria-hidden': !isActive,
'role': 'tabpanel',
'role': 'menu',
'id': subId
});
});
Expand Down

0 comments on commit 163bb73

Please sign in to comment.