Skip to content

E
edx-platform
Commit 8688edaf by Ahsan Ulhaq Committed by GitHub
Options

Merge pull request #14595 from edx/ahsan/ECOM-7312-lms-user-menu-dropdown-a11y-issues 

LMS user menu dropdown a11y issues
parents 7ad0e27d c3d77129
Showing with 5 additions and 2 deletions
lms/templates/user_dropdown.html
...@@ -54,8 +54,11 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_ ...@@ -54,8 +54,11 @@ from openedx.core.djangoapps.user_api.accounts.image_helpers import get_profile_
</a> </a>
</li> </li>
<li class="primary"> <li class="primary">
<div role="group" aria-label="User menu" class="user-menu"> <div class="user-menu">
<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" aria-haspopup="true">
<span class="sr">${_("More options")}</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" class="dropdown-menuitem">${_("Sign Out")}</a></li> <li class="item"><a href="${reverse('logout')}" role="menuitem" class="dropdown-menuitem">${_("Sign Out")}</a></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