Skip to content

fix(footer): align legal link hover color with adjacent nav#8202

Merged
alexander-akait merged 2 commits intowebpack:mainfrom
Sahilshrma31:fix/footer-link-hover-consistency
Apr 22, 2026
Merged

fix(footer): align legal link hover color with adjacent nav#8202
alexander-akait merged 2 commits intowebpack:mainfrom
Sahilshrma31:fix/footer-link-hover-consistency

Conversation

@Sahilshrma31
Copy link
Copy Markdown
Contributor

Summary

This PR fixes the hover state for the footer legal links so they match the hover color used by the adjacent page navigation links, while keeping the existing default blue link color unchanged.

What kind of change does this PR introduce?

fix

Did you add tests for your changes?

No new tests were added.

Does this PR introduce a breaking change?

No.

If relevant, what needs to be documented once your changes are merged or what have you already documented?

No additional documentation is needed.

Use of AI

no.

before

Screen.Recording.2026-04-20.at.3.27.21.AM.mov

after

Screen.Recording.2026-04-20.at.3.27.41.AM.mov

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-js-org Ready Ready Preview, Comment Apr 20, 2026 5:12pm

Request Review

@alexander-akait
Copy link
Copy Markdown
Member

Please use only tailwind utils, no new classes

@Sahilshrma31
Copy link
Copy Markdown
Contributor Author

Sahilshrma31 commented Apr 20, 2026

Please use only tailwind utils, no new classes

@alexander-akait Addressed: removed the custom css and updated this to use tailwind utilities only.

@alexander-akait
Copy link
Copy Markdown
Member

/cc @evenstensberg can you look at this?

@alexander-akait alexander-akait merged commit 85d4e24 into webpack:main Apr 22, 2026
9 checks passed
@Sahilshrma31 Sahilshrma31 deleted the fix/footer-link-hover-consistency branch April 23, 2026 08:22
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