-
Notifications
You must be signed in to change notification settings - Fork 10
Closed
Labels
accessibilitygood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Description
Before You Submit
- I have searched for duplicate or closed issues
- I have read the contributing guidelines
Describe the issue
The mobile menu toggle <button> has aria-expanded and aria-controls attributes, but does not include an aria-label.
Including aria-label="Toggle menu" will help screen readers better describe the button’s function, improving accessibility.
Current code snippet:
<button
aria-expanded={open}
aria-controls="mobile-menu"
onClick={() => setOpen(!open)}
className="block py-3 text-xl text-grey lg:hidden"
>Recommended update:
<button
aria-expanded={open}
aria-controls="mobile-menu"
aria-label="Toggle menu"
onClick={() => setOpen(!open)}
className="block py-3 text-xl text-grey lg:hidden"
>Corresponding file: Header.tsx
Note: This issue is a great starting point for new contributors and is marked as a "good first issue." If you’ve contributed before, please consider looking for other issues without this label.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
accessibilitygood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed