Skip to content

Remove underline on Header links - #83

Merged
danielguillan merged 4 commits into
mainfrom
danielguillan/remove-underlines-header
Jan 21, 2026
Merged

Remove underline on Header links#83
danielguillan merged 4 commits into
mainfrom
danielguillan/remove-underlines-header

Conversation

@danielguillan

@danielguillan danielguillan commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Fix underlines appearing on header links (SiteTitle and navigation links) that were introduced as a side effect of upgrading the @primer/react dependency. Increases CSS selector specificity to ensure text-decoration: none is applied correctly.

Before After
screenshot-W3z5O4Hm-000678@2x screenshot-ACJh3Ons-000677@2x

Copilot AI review requested due to automatic review settings January 21, 2026 10:19
@danielguillan
danielguillan requested a review from a team as a code owner January 21, 2026 10:19

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 fixes underlines appearing on header links after a @primer/react dependency upgrade by increasing CSS selector specificity. The changes ensure that text-decoration: none is properly applied to all header navigation links.

Changes:

  • Increased CSS selector specificity for header site title and navigation links by adding the a element selector
  • Increased CSS selector specificity for dropdown menu links similarly
  • Added changeset documenting the fix

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
packages/theme/components/layout/header/Header.module.css Updated .Header__siteTitle and .Header__link selectors to include a element for higher specificity
packages/theme/components/layout/links-dropdown/LinksDropdown.module.css Updated .link selector to include a element for higher specificity
.changeset/old-fishes-suffer.md Added changeset documenting the removal of underlines from header links

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

Comment thread packages/theme/components/layout/header/Header.module.css Outdated
Comment thread packages/theme/components/layout/links-dropdown/LinksDropdown.module.css Outdated
Comment thread packages/theme/components/layout/header/Header.module.css Outdated
Comment thread packages/theme/components/layout/header/Header.module.css Outdated
@danielguillan
danielguillan merged commit 3c3b1a9 into main Jan 21, 2026
10 checks passed
@primer primer Bot mentioned this pull request Jan 21, 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