Skip to content

[A11Y] Add underlines to text links - #80

Merged
danielguillan merged 3 commits into
mainfrom
danielguillan/a11y-add-link-underline
Jan 7, 2026
Merged

[A11Y] Add underlines to text links#80
danielguillan merged 3 commits into
mainfrom
danielguillan/a11y-add-link-underline

Conversation

@danielguillan

Copy link
Copy Markdown
Contributor

Towards https://github.com/github/accessibility-audits/issues/14451. Adds text-decoration: underline to anchor links.

Before After
screenshot-aFaNY0tH-000651@2x screenshot-sr2q4pKH-000650@2x

@danielguillan
danielguillan requested a review from a team as a code owner December 19, 2025 16:08
Copilot AI review requested due to automatic review settings December 19, 2025 16:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves accessibility of anchor links in prose content by adding underlines to make them more distinguishable from regular text. This addresses accessibility audit issue #14451.

  • Changed text-decoration from none to underline for anchor links in prose content

Reviewed changes

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

File Description
packages/theme/css/prose.module.css Updates link styling to add underline decoration for improved accessibility
.changeset/warm-months-trade.md Documents the patch-level change for the doctocat-nextjs package

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

Comment thread packages/theme/css/prose.module.css Outdated
@danielguillan danielguillan changed the title Danielguillan/a11y add link underline [A11Y] Add underlines to text links Dec 19, 2025
@danielguillan
danielguillan merged commit 772d8de into main Jan 7, 2026
10 checks passed
@danielguillan
danielguillan deleted the danielguillan/a11y-add-link-underline branch January 7, 2026 10:26
@primer primer Bot mentioned this pull request Jan 7, 2026
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.

3 participants