-
Notifications
You must be signed in to change notification settings - Fork 448
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
Set the current page on the top menu #5199
Comments
@Vitaliy-1 are you currently working on some issue related to this request for marking the current page on top menu? |
Hi @jardakotesovec ,
Since the " about " element is not a link to anywhere but a menu opener, it should be changed from See the W3C website menu . It opens, closes, showing an icon change and highlight color to show when active. |
Hi @jardakotesovec , here are the PRs to OMP and OPS 3.4: |
Our accessibility audit recommended to include a visual status on the top menu for the current page.
Currently it is not visible from the menu which page or section the user is viewing.
On that note, it is recommended to enable on the menu the
aria-current="page"
on link item menu for screenreaders being able to announce the current page link to the user.The text was updated successfully, but these errors were encountered: