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

a11y: add a "skip to main content" link #5

Merged
merged 1 commit into from
Sep 29, 2023
Merged

a11y: add a "skip to main content" link #5

merged 1 commit into from
Sep 29, 2023

Conversation

2kool4idkwhat
Copy link

@2kool4idkwhat 2kool4idkwhat commented Jul 12, 2023

The theme already has great accessibility but it could be improved even further with a "skip to main content" link. Basically, these skiplinks give people with screen readers and people who navigate websites only with their keyboards an easy way to skip the stuff that gets repeated on every page (in this case the title and navigation links in the header), so that they can get to the content right away

For more detailed overview you can see this if you want to: https://webaim.org/techniques/skipnav/

The skiplink in my pull request is visually hidden until it's focused so people who don't need it won't even know it's there

@clente clente self-requested a review September 29, 2023 22:35
@clente clente self-assigned this Sep 29, 2023
@clente clente added the enhancement New feature or request label Sep 29, 2023
@clente
Copy link
Owner

clente commented Sep 29, 2023

Sorry it took me such a long time to get to your PR... For some weird reason I wasn't watching my own repo 😅

Thank you very much for the contribution, I had never heard of skiplinks before! I'll accept the PR and then make a few changes just so users are able to translate "Skip to main content" to their websites' languages.

@clente clente merged commit 6f2e7d0 into clente:main Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants