Skip to content

Restores underlines to links in the content#3656

Merged
ahpook merged 1 commit into
github:mainfrom
aardrian:patch-1
May 11, 2026
Merged

Restores underlines to links in the content#3656
ahpook merged 1 commit into
github:mainfrom
aardrian:patch-1

Conversation

@aardrian
Copy link
Copy Markdown
Contributor

@aardrian aardrian commented May 8, 2026

/assets/css/index.css has this code, which removes underlines from all links on the page:

a {
  color: #0366d6;
  text-decoration: none;
}

I cannot find that file in the repo, so I've made my change to the custom.scss file.

This update reverts the underline removal for content links. Otherwise the links fail SC 1.4.1 Use of Color because, while the underline-on-hover style elsewhere suggests someone was trying to pass it via Technique G183, the contrast ratio between the blue link and black text is only 2.7:1.

This change fixes the WCAG violation and also provides demonstrably more usable links ("demonstrably" because I can point to research if needed).

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


index.css has this code, which removes underlines from _all_ links on the page:
```
a {
  color: #0366d6;
  text-decoration: none;
}
```
This update reverts that for content links.
@mgifford
Copy link
Copy Markdown

mgifford commented May 8, 2026

Thanks for reporting it here.

Copy link
Copy Markdown
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

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

This style is computed from the upstream Primer design system's primer-base module, which is why you didn't see it in the repo. A fix should probably go upstream, but we're pinned to a very old version of Primer for this site so this seems like a good workaround.

@ahpook ahpook added this pull request to the merge queue May 11, 2026
Merged via the queue into github:main with commit 443bf3b May 11, 2026
7 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.

3 participants