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

Accessibility issue in the header #2584

Closed
SalarGhotaslo opened this issue Mar 30, 2022 · 2 comments
Closed

Accessibility issue in the header #2584

SalarGhotaslo opened this issue Mar 30, 2022 · 2 comments
Labels
🐛 bug Something isn't working the way it should (including incorrect wording in documentation)

Comments

@SalarGhotaslo
Copy link

Description of the issue

The .govuk-skip-link in the template component is causing an accessibility issue that states page content is not contained by landmarks.

Steps to reproduce the issue

On the https://design-system.service.gov.uk/ if you inspect the page then --> go on Axe dev tools --> Scan page --> then you'll see something similar to the screenshot below. You can then highlight the issue by clicking on the All page content should be contained by landmarks and then clicking the highlight.

image

Actual vs expected behavior

I would expect there to be no accessibility issue.

Environment (where applicable)

The issue can be seen on the https://design-system.service.gov.uk/ webpage so it is on the latest one.

We found it happen when trying to add the cookie banner component into our project, it seemed to then expose this issue. We can see that this issue wouldn't happen if the govuk-skip-link was inside the header component rather than inside the template component.

  • Operating system:
  • Browser:
  • Browser version:
  • GOV.UK Frontend Version:
@SalarGhotaslo SalarGhotaslo added awaiting triage Needs triaging by team 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) labels Mar 30, 2022
@querkmachine
Copy link
Member

Hi @SalarGhotaslo.

I'm going to close this as it appears to be a duplicate of #1604.

Interestingly, documentation from Deque (who make Axe) seems to indicate that skip links do not have to be within a landmark region, so no changes may be necessary here. (Emphasis mine.)

It is best practice to contain all content excepting skip links, within distinct regions such as the header, nav, main, and footer.

Feel free to follow up on #1604 if you have more information to share.

@querkmachine querkmachine removed the awaiting triage Needs triaging by team label Mar 30, 2022
@SalarGhotaslo
Copy link
Author

Thanks for the prompt response! Will have a read

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working the way it should (including incorrect wording in documentation)
Projects
None yet
Development

No branches or pull requests

2 participants