-
Notifications
You must be signed in to change notification settings - Fork 3
Skip to main content link is missing #1431
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
Labels
Comments
thatbudakguy
added a commit
to projectblacklight/blacklight
that referenced
this issue
Oct 9, 2024
Some accessibility checkers, like SiteImprove, regard it as a failure that the 'skip to main' link is not the first focusable element on the page. This swaps the order of the 'skip to main' and 'skip to search' links so that 'skip to main' comes first, which is in line with the WCAG 2.2 guidelines (https://www.w3.org/WAI/WCAG22/Techniques/general/G1.html). Related: sul-dlss/earthworks#1431
isn't the search the first element in the main content? Won't skipping to main require extra keystrokes? |
It looks like there's an added link now, "Skip to first result," so the order should be "Skip to main content" (required), "Skip to first result," and then "Skip to search." We discussed this with SODA and at a DLUX team meeting. Here's the reasoning:
|
cbeer
pushed a commit
to projectblacklight/blacklight
that referenced
this issue
Oct 23, 2024
Some accessibility checkers, like SiteImprove, regard it as a failure that the 'skip to main' link is not the first focusable element on the page. This swaps the order of the 'skip to main' and 'skip to search' links so that 'skip to main' comes first, which is in line with the WCAG 2.2 guidelines (https://www.w3.org/WAI/WCAG22/Techniques/general/G1.html). Related: sul-dlss/earthworks#1431
cbeer
pushed a commit
to projectblacklight/blacklight
that referenced
this issue
Oct 23, 2024
Some accessibility checkers, like SiteImprove, regard it as a failure that the 'skip to main' link is not the first focusable element on the page. This swaps the order of the 'skip to main' and 'skip to search' links so that 'skip to main' comes first, which is in line with the WCAG 2.2 guidelines (https://www.w3.org/WAI/WCAG22/Techniques/general/G1.html). Related: sul-dlss/earthworks#1431
This will be fixed by updating Blacklight |
thatbudakguy
added a commit
that referenced
this issue
Oct 30, 2024
thatbudakguy
added a commit
that referenced
this issue
Oct 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
SiteImprove flags this as "Skip to main content is missing." It's not missing, but it's not first. "Skip to search" is first instead. See screenshot.
Guidance
https://www.w3.org/WAI/WCAG22/Techniques/general/G1.html.
Given the guidance linked above (which suggests it should be first), I recommend that we reverse the order "Skip to main content" and then "Skip to search."
The text was updated successfully, but these errors were encountered: