Skip to content

Commit 82c1728

Browse files
committed
fix: imporve accessibility on main_links_mobile_account
1 parent 81e861e commit 82c1728

File tree

1 file changed

+15
-1
lines changed

1 file changed

+15
-1
lines changed

decidim-core/app/views/layouts/decidim/header/_main_links_mobile_account.html.erb

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,9 @@
1010
</div>
1111

1212
<div>
13-
<div class="main-bar__links-mobile__trigger" onclick="document.querySelector('#dropdown-trigger-links-mobile').click();document.querySelector('#dropdown-menu-account-mobile').removeAttribute('aria-modal')">
13+
<div class="main-bar__links-mobile__trigger" tabindex="0" onClick="document.querySelector('#dropdown-trigger-links-mobile').click();document.querySelector('#dropdown-menu-account-mobile').removeAttribute('aria-modal')">
1414
<%= icon "close-line" %>
15+
<p class="sr-only"><%= t("close", scope: "decidim.shared.flag_modal") %></p>
1516
</div>
1617
</div>
1718
</div>
@@ -37,3 +38,16 @@
3738
</ul>
3839
</div>
3940
</div>
41+
<script type="text/javascript">
42+
const closeDiv = document.querySelector('div.main-bar__links-mobile__trigger');
43+
const menuDropdown = document.querySelector('#dropdown-menu-account-mobile');
44+
const dropdownTrigger = document.querySelector('#dropdown-trigger-links-mobile');
45+
// 32 is code for space bar and 13 is code for enter
46+
closeDiv.addEventListener('keydown', function(e){
47+
if (e.keyCode === 13 || e.keyCode === 32) {
48+
menuDropdown.removeAttribute('aria-modal');
49+
menuDropdown.setAttribute('aria-hidden', "true");
50+
dropdownTrigger.focus();
51+
}
52+
})
53+
</script>

0 commit comments

Comments
 (0)