Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Mobile navigation missing current page attribute #48

Closed
djmtype opened this issue Dec 21, 2022 · 1 comment
Closed

Mobile navigation missing current page attribute #48

djmtype opened this issue Dec 21, 2022 · 1 comment
Assignees
@markteekman
Labels
bug Something isn't working

Comments

@djmtype
Copy link

djmtype commented Dec 21, 2022

I wasn't sure if there was a reason, but aria-current=page is not being applied to the applicable mobile navigation links.
@markteekman
Copy link
Member

Hey @djmtype, thanks again for your sharp eye! I think it's due to the fact that the theme uses two menu's for the automatic responsive toggle behavior 🤔 I'll look into it!

@markteekman markteekman self-assigned this Jan 10, 2023
@markteekman markteekman added the bug Something isn't working label Jan 10, 2023
markteekman added a commit that referenced this issue Jan 21, 2023
@markteekman
Fix aria-current not being set for the mobile menu #48
a0fc1f1
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@markteekman markteekman

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@djmtype @markteekman