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

Add additional skip link on guide views #1851

Merged
merged 1 commit into from
Sep 10, 2020

Conversation

owenatgov
Copy link
Contributor

What

Adds an additional skip link above the guide contents that goes to the actual content of the guide.

Why

This was identified as a WCAG fail (specifically WCAG SC 2.4.1) as for keyboard only users, it meant that they had the option to skip navigation in the govuk header but still have to tab through the navigation of the guide contents to get to the "actual" content of the guide. This solution of using an additional skip link is used on govuk search pages to skip past filters.

Card: https://trello.com/c/x3QKWP0z/340-cannot-bypass-repeated-navigation

⚠️ This application is Continuously Deployed: ⚠️

  • Merged changes are automatically deployed to staging and production.

  • Make sure you follow the guidance for deployments before you merge.

  • Check your branch is being deployed in the Release app, after merging.

@bevanloon bevanloon temporarily deployed to government-f-additional-2gtkep September 8, 2020 15:54 Inactive
Copy link
Contributor

@injms injms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small visual discrepancy that a bit of margin will sort out - but nothing that stops this from being a solid improvement over not having a skip link at all.

@injms injms merged commit e727bc6 into master Sep 10, 2020
@injms injms deleted the additional-skip-link-on-guide-pages branch September 10, 2020 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants