fix(ui): update footer social icons to darken on hover#325
fix(ui): update footer social icons to darken on hover#325krishikajain28 wants to merge 3 commits intokeploy:mainfrom
Conversation
Signed-off-by: Krishika Jain <krishikajain28@gmail.com>
3fdc8f8 to
a9c0a10
Compare
Code Review SummaryStatus: No Issues Found | Recommendation: Merge OverviewThis PR improves the footer's social media link hover states by:
The changes are straightforward CSS improvements with no functional impact. Files Reviewed (1 file)
|
|
Hi @krishikajain28 Looking forward to seeing more from you! |
|
Hey @krishikajain28 👋 — thanks so much for contributing to the project, really appreciate it! Your PR looks great and has been marked for merging. Here's a quick note from the reviewer:
We'll get this merged in soon. Keep the great work coming! 🚀 |
There was a problem hiding this comment.
Pull request overview
Updates footer social icon link styling so icons darken on hover, matching the hover behavior of the footer text links.
Changes:
- Changed footer social links from
hover:text-gray-500tohover:text-gray-900. - Added
duration-300to smooth the hover color transition on social icons.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div className="max-lg:ml-10"> | ||
| <div className="mr-4 shrink-0 w-52"> | ||
| <Link href="https://keploy.io/"> | ||
|
|
There was a problem hiding this comment.
There’s an extra blank line containing whitespace inside the <Link> before the <Image>; this adds trailing whitespace/noise and may trip formatting/lint rules. Please remove the whitespace-only line.
| </Link> | ||
| </div> | ||
| <div className="flex space-x-6 my-4"> | ||
|
|
There was a problem hiding this comment.
There’s an extra whitespace-only line at the start of the social links container. Please remove the blank line to keep the JSX clean and avoid trailing whitespace.
Description
I noticed that the social media icons in the footer (Twitter, GitHub, etc.) were staying gray when hovered over, but the text links were turning black.
I updated the classes to
hover:text-gray-900 transition-colors duration-300so the icons now darken on hover, matching the rest of the footer links.Type of Change
Testing
localhost:3000/blog).Demo
Checklist