feat(a11y): Add skip-to-content accessibility link#17527
Merged
rahulchhabria merged 3 commits intomasterfrom Apr 29, 2026
Merged
feat(a11y): Add skip-to-content accessibility link#17527rahulchhabria merged 3 commits intomasterfrom
rahulchhabria merged 3 commits intomasterfrom
Conversation
Add a keyboard-accessible "Skip to content" link that appears on Tab focus and jumps to the main content area. Resets focus navigation on client-side route changes so the link remains the first Tab stop.
Add matching #main targets to the home and 404 pages so the root skip link works outside DocPage-rendered routes. Remove the unused skip-link anchor ref.
Merge origin/master into the skip-to-content branch. Keep the incoming home page card updates while preserving the #main target needed by the skip link.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Author
|
@macuzi - this should be good to go. can you give it a review pls? @sfanahata -- i might need you to stamp it too |
sfanahata
approved these changes
Apr 29, 2026
Contributor
sfanahata
left a comment
sfanahata
left a comment
There was a problem hiding this comment.
Looks good! Thanks for adding this.
Contributor
|
Looks good! Tested locally — skip-to-content works well on docs pages. Minor questions/notes for potential future iteration:
Neither is a blocker and looks great, approving! Thanks for the assistance 🙂 |
Contributor
Honestly, I think there is definitely more we can do about tabbing through the site, but chewing on what would be best to improve after this. One annoying factor is the left nav losing focus after selecting a page. Not sure there's much we can do, but I might look into it. Looks like that |
Contributor
|
Agreed on both fronts @sfanahata 🙂 |
cameroncooke
pushed a commit
that referenced
this pull request
Apr 30, 2026
## DESCRIBE YOUR PR Adds a root-level skip-to-content link and ensures non-DocPage routes have a matching `#main` target. This keeps keyboard users from getting stuck on the home page or 404 page when using the skip link. This replaces fork PR #17479 so the Codeowner assignment workflow can run with same-repo permissions. ## IS YOUR CHANGE URGENT? - [ ] Urgent deadline (GA date, etc.): - [ ] Other deadline: - [x] None: Not urgent, can wait up to 1 week+ ## SLA - Teamwork makes the dream work, so please add a reviewer to your PRs. - Please give the docs team up to 1 week to review your PR unless you have added an urgent due date to it. Thanks in advance for your help! ## PRE-MERGE CHECKLIST - [ ] Checked Vercel preview for correctness, including links - [x] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) --------- Co-authored-by: macuzi <lamberto.asghedom@sentry.io>
coolguyzone
pushed a commit
that referenced
this pull request
May 1, 2026
## DESCRIBE YOUR PR Adds a root-level skip-to-content link and ensures non-DocPage routes have a matching `#main` target. This keeps keyboard users from getting stuck on the home page or 404 page when using the skip link. This replaces fork PR #17479 so the Codeowner assignment workflow can run with same-repo permissions. ## IS YOUR CHANGE URGENT? - [ ] Urgent deadline (GA date, etc.): - [ ] Other deadline: - [x] None: Not urgent, can wait up to 1 week+ ## SLA - Teamwork makes the dream work, so please add a reviewer to your PRs. - Please give the docs team up to 1 week to review your PR unless you have added an urgent due date to it. Thanks in advance for your help! ## PRE-MERGE CHECKLIST - [ ] Checked Vercel preview for correctness, including links - [x] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) --------- Co-authored-by: macuzi <lamberto.asghedom@sentry.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DESCRIBE YOUR PR
Adds a root-level skip-to-content link and ensures non-DocPage routes have a matching
#maintarget. This keeps keyboard users from getting stuck on the home page or 404 page when using the skip link.This replaces fork PR #17479 so the Codeowner assignment workflow can run with same-repo permissions.
IS YOUR CHANGE URGENT?
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST