Skip to content

Add skip link for keyboard accessibility#15

Open
web3blind wants to merge 2 commits into
TheCodeVerseHub:masterfrom
web3blind:accessibility-skip-link
Open

Add skip link for keyboard accessibility#15
web3blind wants to merge 2 commits into
TheCodeVerseHub:masterfrom
web3blind:accessibility-skip-link

Conversation

@web3blind
Copy link
Copy Markdown

Summary

Adds a skip-to-content link so keyboard and screen-reader users can bypass repeated navigation and jump directly to the main content.

What changed

  • Added a Skip to main content link in the root layout.
  • The link is visually hidden until focused.
  • Added id="main-content" and tabIndex={-1} to the main landmarks on key pages.

Why

This addresses #13 and supports WCAG 2.4.1 Bypass Blocks by giving users a reliable way to skip repeated navigation.

Validation

  • Ran targeted ESLint on the changed files.
  • No new errors were introduced in the changed files.
  • Note: the full project lint currently reports pre-existing unrelated issues in other files.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 11, 2026

👷 Deploy request for thecodeversehub pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit c4731eb

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a global “Skip to main content” link to improve keyboard/screen-reader navigation (WCAG 2.4.1 Bypass Blocks), and wires up skip targets on several primary pages.

Changes:

  • Added a visually-hidden-until-focus skip link in RootLayout pointing to #main-content.
  • Added id="main-content" and tabIndex={-1} to <main> on /, /pages, /resources, and /ban-appeal.
  • Wrapped the home page content in a <main> landmark to serve as the skip target.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/app/layout.tsx Adds global skip-link anchor at the top of <body>.
src/app/page.tsx Wraps the home page’s primary content in <main id="main-content">.
src/app/pages/page.tsx Marks the docs index <main> as the skip target.
src/app/resources/page.tsx Marks the resources content <main> as the skip target.
src/app/ban-appeal/page.tsx Marks the ban appeal form <main> as the skip target.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/app/layout.tsx
Comment on lines +118 to +123
<a
href="#main-content"
className="sr-only focus:not-sr-only focus:fixed focus:left-4 focus:top-4 focus:z-[100] focus:rounded-md focus:bg-white focus:px-4 focus:py-2 focus:text-sm focus:font-semibold focus:text-black focus:outline-none focus:ring-2 focus:ring-violet-500"
>
Skip to main content
</a>
@web3blind
Copy link
Copy Markdown
Author

Addressed the skip-link target feedback by adding id="main-content" / tabIndex={-1} to the dynamic docs page template and not-found content, so the global skip link has a target on those routes too.\n\nValidation:\n- npm run build passes.\n- npm run lint still reports existing unrelated lint issues in untouched components (About, TechMarquee, TerminalWindow, TextPressure) plus existing warnings.

@web3blind web3blind force-pushed the accessibility-skip-link branch from 64182c0 to c4731eb Compare June 1, 2026 16:57
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.

2 participants