Angular staff navbar needs ARIA attribute to indicate the current page
Bug #2046820 reported by
Stephanie Leary
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Low
|
Unassigned |
Bug Description
In the Angular staff main navigation menu, there is nothing to indicate which page is current. We need to add the aria-current attribute for accessibility compliance, and we can style the link differently as well if we want. We can use the RouterLinkActive directive: https:/
For the ARIA attribute, we will add this to each menu item link that uses routerLink:
routerLinkActiv
This will not work on non-Angular pages, so there is no need to add these to links that use href.
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Here's a branch: https:/ /git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ sleary/ LP2046820- angular- nav-aria- current
To test:
1. Open up an Angular screen that is listed somewhere in the main navigation, like staff/catalog/ search.
2. Inspect the menu link that corresponds to your current page.
3. See if the <a> tag includes the aria-current="page" attribute, and is highlighted in pale green.
Note that this is only for Angular screens.