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.

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

Defect 5 - Critical - Skip to Main Content #19375

Closed
1 task done
Tracked by #19915 ...
srancour opened this issue Oct 1, 2024 · 1 comment · Fixed by #20094
Closed
1 task done
Tracked by #19915 ...

Defect 5 - Critical - Skip to Main Content #19375

srancour opened this issue Oct 1, 2024 · 1 comment · Fixed by #20094
Assignees
Labels
508-audit Issues associated with the 508 office audits a11y-defect-1 Critical accessibility issue that should be fixed in the next sprint accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops

Comments

@srancour
Copy link
Contributor

srancour commented Oct 1, 2024

Description or Additional Context

9/16/2024: The skip to main content link should take the user to main content of the page which is usually the H1 tag, but can be some other place in the content. The Skip to Main Content link always goes to the Manage link which is the first element in the block of content you should be skipping.

Example

The "Skip to main content" on every page seems to always set the url to #undefined instead of the #main-content it's actually set to so the focus always goes to the next element instead of the main content on the page. Screenshot of upper portion of chrome with the url showing #undefined at the end and the Skip to main content link is visible and selected

Code snippet

Skip to main content link <a href="#main-content" class="visually-hidden focusable">Skip to main content</a>
Main content tag <div class="visually-hidden"><a id="main-content" tabindex="-1"></a></div>

Steps to reproduce

Navigate: Home or any other page
Tab on load to show the "Skip to main content" link, press link, tab to next element and focus moves to the "Manage" menu element.

Recommended resolution

This seems to be a similar issue to #19368 and fixing one could fix both.

Defect Criterion

WCAG 2.4.1 - Bypass Blocks - A mechanism is available to bypass blocks of content that are repeated on multiple Web pages.

Users affected

  • Keyboard only
  • Screen reader

Teams that this affects:

  • CMS Team
@srancour srancour added 508-audit Issues associated with the 508 office audits a11y-defect-1 Critical accessibility issue that should be fixed in the next sprint accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops Needs refining Issue status labels Oct 1, 2024
@srancour srancour changed the title Defect 5 Skip to Main Content Defect 5 - Critical - Skip to Main Content Oct 7, 2024
@srancour
Copy link
Contributor Author

srancour commented Nov 5, 2024

This also seems to have to be related with work that the last team had been doing as well in #12145.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
508-audit Issues associated with the 508 office audits a11y-defect-1 Critical accessibility issue that should be fixed in the next sprint accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
3 participants