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
Contributor

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!

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.

2 participants