Skip to content

A11y/skip link#694

Closed
gcamacho079 wants to merge 2 commits intomainfrom
a11y/skip-link
Closed

A11y/skip link#694
gcamacho079 wants to merge 2 commits intomainfrom
a11y/skip-link

Conversation

@gcamacho079
Copy link
Contributor

Description

Adds a skip link to the main content, and moves focus to the top of the page on route changes

Related issues

@gcamacho079 gcamacho079 requested a review from a team as a code owner March 19, 2025 19:13
@netlify
Copy link

netlify bot commented Mar 19, 2025

Deploy Preview for craft-docs ready!

Name Link
🔨 Latest commit f272068
🔍 Latest deploy log https://app.netlify.com/sites/craft-docs/deploys/67db17699e70730008da4942
😎 Deploy Preview https://deploy-preview-694--craft-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@gcamacho079
Copy link
Contributor Author

Closing because I noticed the route change is also triggered on hash updates, and the focus is then reset to the top unnecessarily. Will fix!

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.

1 participant