Skip to content

a11y: add skip-to-content link - #265

Merged
yakew7 merged 1 commit into
StudentSuite:mainfrom
propcgamer20-png:a11y-skip-to-content
Sep 8, 2026
Merged

a11y: add skip-to-content link#265
yakew7 merged 1 commit into
StudentSuite:mainfrom
propcgamer20-png:a11y-skip-to-content

Conversation

@propcgamer20-png

Copy link
Copy Markdown
Contributor

Adds a visually-hidden-until-focused 'Skip to content' link as the first focusable element in the body, jumping to a new id="main-content" on

. Follows the same sr-only/focus:not-sr-only Tailwind pattern already used elsewhere in the codebase (dialog.tsx, sheet.tsx close buttons).

Fixes #261

What this changes

Type of change

  • New public place(s)
  • New or updated benefit guide
  • New or updated resource link
  • Code or fix
  • Docs

Proof (required for new public places)

Fill this in for every place added in this PR. It stays in the PR, not in the
dataset.

Field Value
Source or citation
Google Maps rating (must be >= 4.0)
Google Maps review count (must be >= 50)
Date verified
  • Each place's rating is 4.0 or higher.
  • Each place has 50 or more reviews.
  • Coordinates point to the correct entrance.
  • The JSON record stops at gmaps_link and added_by (no proof fields committed).

Checklist

  • The site hosts no copyrighted files; resource and paper changes are links only.
  • No em dashes in any copy.

Adds a visually-hidden-until-focused 'Skip to content' link as the
first focusable element in the body, jumping to a new id="main-content"
on <main>. Follows the same sr-only/focus:not-sr-only Tailwind pattern
already used elsewhere in the codebase (dialog.tsx, sheet.tsx close
buttons).

Fixes StudentSuite#261
@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@propcgamer20-png is attempting to deploy a commit to the studentsuite Team on Vercel.

A member of the Team first needs to authorize it.

@yakew7
yakew7 merged commit 575761c into StudentSuite:main Sep 8, 2026
4 of 5 checks passed
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.

a11y: no skip-to-content link — keyboard users tab through the full navbar every page

2 participants