Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Navcurrent: Add aria-current="page" attribute #9307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Navcurrent: Add aria-current="page" attribute #9307

wants to merge 1 commit into from

Commits on Mar 11, 2022

  1. Navcurrent: Add aria-current="page" attribute 

    The navcurrent plugin is a perfect fit for the aria-current attribute. The former adds a class ("wb-navcurr") to visually-denote links to the current page, whereas the latter represents it in semantics.

Scenarios this covers:
* Any links to the current page the plugin tries adding the "wb-navcurr" class to

Scenarios this won't cover:
* Top-level mega menu links that point to dropdown IDs (aria-current is inappropriate in those scenarios)
    @EricDunsworth
    EricDunsworth committed Mar 11, 2022
    Configuration menu
    Copy the full SHA
    1bf973f View commit details
    Browse the repository at this point in the history