-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(menu): add aria-current - FRONT-4405 #3372
Conversation
emeryro
commented
May 2, 2024
•
edited
Loading
edited
- add aria-current=page to identify the current page in the menu
- add aria-current=true to identify selected menu item
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mmhh..if we are about to do so, we need to demo only one current item, otherwise we would be setting the current page multiple times on different links which would not be ok, but we should also update the docs related to this parameter, in the mega-menu i used this: - "is_current" (boolean) Whether the item links to the current page
The markup has been updated to use |