search

ucsc / ucsc-2022

Official WordPress theme for UC Santa Cruz
6 stars 0 forks source link

Adjust HTML in a11y skip link #293

Closed knice closed 4 months ago

knice commented 6 months ago

https://ucsc.service-now.com/now/nav/ui/classic/params/target/incident.do%3Fsys_id%3D2df010581b93f510931886e0604bcbca%26sysparm_stack%3D%26sysparm_view%3D

knice commented 5 months ago

The skip link is in functions.php. It was added by Modern Tribe. I think it is getting flagged because it is just an a tag which is an inline element. The link needs a block-level wrapper element with a proper ARIA role to make it a landmark role (landmark roles are the only roles that can be direct descendants of the body tag).

You can refer to the skip link functionality we have on all WCMS pages for reference.

knice commented 4 months ago

Additionally, the skip link nav was being added just above the page content wrapper. A11y testing tools flagged that as an issue. The skip link should be as close to the top of the page as possible.