Commit c2c17c11 by Jareer Ahsan

Added classname to find menu links

parent 4b8490d9
...@@ -8,11 +8,11 @@ ...@@ -8,11 +8,11 @@
// define variables for code legibility // define variables for code legibility
var $dropdownMenuToggle = $('.dropdown'); var $dropdownMenuToggle = $('.dropdown');
var $dropdownMenu = $('.dropdown-menu'); var $dropdownMenu = $('.dropdown-menu');
var menuItems = $dropdownMenu.find('a'); var menuItems = $dropdownMenu.find('.dropdown-menuitem');
// bind menu toggle click for later use // bind menu toggle click for later use
$dropdownMenuToggle.toggle(function() { $dropdownMenuToggle.toggle(function() {
$dropdownMenu.addClass('expanded').find('a').first() $dropdownMenu.addClass('expanded').find('.dropdown-menuitem').first()
.focus(); .focus();
$dropdownMenuToggle.addClass('active').attr('aria-expanded', 'true'); $dropdownMenuToggle.addClass('active').attr('aria-expanded', 'true');
}, function() { }, function() {
......
...@@ -27,11 +27,11 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_ ...@@ -27,11 +27,11 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_
</button> </button>
<ul class="dropdown-menu list-divided is-hidden" id="${_("Usermenu")}" tabindex="-1"> <ul class="dropdown-menu list-divided is-hidden" id="${_("Usermenu")}" tabindex="-1">
<%block name="navigation_dropdown_menu_links" > <%block name="navigation_dropdown_menu_links" >
<li class="dropdown-item item has-block-link"><a href="${reverse('dashboard')}" class="action">${_("Dashboard")}</a></li> <li class="dropdown-item item has-block-link"><a href="${reverse('dashboard')}" class="action dropdown-menuitem">${_("Dashboard")}</a></li>
<li class="dropdown-item item has-block-link"><a href="${reverse('learner_profile', kwargs={'username': user.username})}" class="action">${_("Profile")}</a></li> <li class="dropdown-item item has-block-link"><a href="${reverse('learner_profile', kwargs={'username': user.username})}" class="action dropdown-menuitem">${_("Profile")}</a></li>
<li class="dropdown-item item has-block-link"><a href="${reverse('account_settings')}" class="action">${_("Account")}</a></li> <li class="dropdown-item item has-block-link"><a href="${reverse('account_settings')}" class="action dropdown-menuitem">${_("Account")}</a></li>
</%block> </%block>
<li class="dropdown-item item has-block-link"><a href="${reverse('logout')}" role="menuitem" class="action">${_("Sign Out")}</a></li> <li class="dropdown-item item has-block-link"><a href="${reverse('logout')}" role="menuitem" class="action dropdown-menuitem">${_("Sign Out")}</a></li>
</ul> </ul>
</div> </div>
</div> </div>
...@@ -53,7 +53,7 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_ ...@@ -53,7 +53,7 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_
<button class="dropdown" aria-expanded="false"><span class="sr">${_("More options dropdown")}</span><span class="fa fa-sort-desc" aria-hidden="true"></span></button> <button class="dropdown" aria-expanded="false"><span class="sr">${_("More options dropdown")}</span><span class="fa fa-sort-desc" aria-hidden="true"></span></button>
<ul class="dropdown-menu" aria-label="More Options" role="menu"> <ul class="dropdown-menu" aria-label="More Options" role="menu">
${navigation_dropdown_menu_links()} ${navigation_dropdown_menu_links()}
<li class="item"><a href="${reverse('logout')}" role="menuitem">${_("Sign Out")}</a></li> <li class="item"><a href="${reverse('logout')}" role="menuitem" class="dropdown-menuitem">${_("Sign Out")}</a></li>
</ul> </ul>
</div> </div>
</li> </li>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment