Skip to content

fix(ui): update footer social icons to darken on hover#325

Open
krishikajain28 wants to merge 3 commits intokeploy:mainfrom
krishikajain28:fix/footer-social-hover
Open

fix(ui): update footer social icons to darken on hover#325
krishikajain28 wants to merge 3 commits intokeploy:mainfrom
krishikajain28:fix/footer-social-hover

Conversation

@krishikajain28
Copy link
Copy Markdown

@krishikajain28 krishikajain28 commented Feb 14, 2026

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-300 so the icons now darken on hover, matching the rest of the footer links.

Type of Change

  • Bug fix (UI consistency)

Testing

  • Verified on local dev environment (localhost:3000/blog).
  • Hovering over icons now correctly changes their color to black.
  • Checked mobile view to ensure no layout breakage.

Demo

image

Checklist

  • My code follows the style guidelines
  • I have performed a self-review
  • Tested locally

Signed-off-by: Krishika Jain <krishikajain28@gmail.com>
@krishikajain28 krishikajain28 force-pushed the fix/footer-social-hover branch from 3fdc8f8 to a9c0a10 Compare February 14, 2026 10:54
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented Feb 25, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR improves the footer's social media link hover states by:

  • Changing hover color from text-gray-500 to text-gray-900 (provides actual visual feedback on hover)
  • Adding duration-300 for smoother transition animations

The changes are straightforward CSS improvements with no functional impact.

Files Reviewed (1 file)
  • components/footer.tsx - CSS hover state improvements for social links

@dhananjay6561
Copy link
Copy Markdown
Member

Hi @krishikajain28
Thanks for the PR and for spotting this!
The change fixes a very minor UI issue. Since the scope of this PR is quite small, we’d recommend exploring a slightly broader improvement or addressing a larger issue and including this fix as part of that update.

Looking forward to seeing more from you!

@dhananjay6561 dhananjay6561 requested a review from Copilot April 6, 2026 12:05
@dhananjay6561
Copy link
Copy Markdown
Member

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:

Real bug fix — hover was text-gray-500 on text-gray-500 base (no-op). Now text-gray-900.

We'll get this merged in soon. Keep the great work coming! 🚀

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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-500 to hover:text-gray-900.
  • Added duration-300 to smooth the hover color transition on social icons.

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

Comment thread components/footer.tsx
<div className="max-lg:ml-10">
<div className="mr-4 shrink-0 w-52">
<Link href="https://keploy.io/">

Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

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.

Suggested change

Copilot uses AI. Check for mistakes.
Comment thread components/footer.tsx
</Link>
</div>
<div className="flex space-x-6 my-4">

Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

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.

Suggested change

Copilot uses AI. Check for mistakes.
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