Skip to content

Commit 39f4932

Browse files
lfranckeclaude
andcommitted
feat(a11y): Give the mobile menu button an accessible name and aria state
Upstream did this in header-content.hbs (b35fcda, 86237c9), a partial this repo replaced with its own header.hbs, so the change could not be ported and is reapplied here. Ours targets the popup-menu container rather than upstream's topbar-nav. The button previously had no accessible name at all, and nothing exposed its open/closed state; 05-mobile-navbar.js already maintains aria-expanded and now reads aria-controls, falling back to data-target. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 67ce09f commit 39f4932

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

ui/src/partials/header.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
{{/navbar}}
2828
</div>
2929
<div id="topbar-right" class="navbar-end">
30-
<button class="navbar-burger" data-target="popup-menu">
30+
<button class="navbar-burger" aria-controls="popup-menu" aria-expanded="false" aria-label="Toggle main menu">
3131
<span></span>
3232
<span></span>
3333
<span></span>

0 commit comments

Comments
 (0)