Skip to content

Commit

Permalink
[4.0] [Validation] Fix backend menu items with role="menuitem" not pl…
Browse files Browse the repository at this point in the history
…aced inside element with role="menu" (#21150)

* Fix menu items with role="menuitem" not placed inside element with role="menubar"

* Use role="menu" instead of role="menubar"
  • Loading branch information
ggppdk authored and laoneo committed Jul 17, 2018
1 parent db28079 commit 065fd0f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion administrator/modules/mod_menu/tmpl/default.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
if ($root->hasChildren())
{
echo '<div class="main-nav-container" role="navigation" aria-label="Main menu">';
echo '<ul id="menu" class="' . $class . '">' . "\n";
echo '<ul id="menu" class="' . $class . '" role="menu">' . "\n";
echo '<li role="menuitem">';
echo '<a id="menu-collapse" href="#">';
echo '<span id="menu-collapse-icon" class="fa-fw fa fa-toggle-off" aria-hidden="true"></span>';
Expand Down

0 comments on commit 065fd0f

Please sign in to comment.