chore: replace Twitter icon with X logo in referrer display #27

Merged
uz1mani merged 1 commits from staging into main 2026-02-12 20:59:56 +00:00
uz1mani commented 2026-02-12 20:59:49 +00:00 (Migrated from github.com)

chore: replace Twitter icon with X logo in referrer display

chore: replace Twitter icon with X logo in referrer display
greptile-apps[bot] commented 2026-02-12 21:01:19 +00:00 (Migrated from github.com)

Greptile Overview

Greptile Summary

Replaced the legacy Twitter bird icon with the modern X logo throughout the referrer display system. The change affects both the icon component and the favicon fallback logic.

Key Changes:

  • Switched from FaTwitter (from react-icons/fa) to FaXTwitter (from react-icons/fa6)
  • Updated icon color from text-blue-400 to neutral colors that adapt to dark mode (text-neutral-800 dark:text-neutral-200)
  • Added REFERRER_USE_X_ICON set containing Twitter-related domains (t.co, x.com, twitter.com, www.twitter.com)
  • Modified getReferrerFavicon to return null for X/Twitter domains, ensuring the custom X icon is always used instead of fetching favicons that might show the old bird logo
  • Updated changelog to document this as version 0.5.1-alpha

Confidence Score: 5/5

  • This PR is safe to merge with no issues
  • The changes are minimal, well-contained, and correctly implemented. The icon replacement is straightforward with proper dark mode support, and the favicon logic ensures consistent X branding across all Twitter-related domains. The changelog follows project conventions.
  • No files require special attention

Important Files Changed

Filename Overview
CHANGELOG.md Added version 0.5.1-alpha entry documenting the X logo icon change with proper changelog formatting and version links
lib/utils/icons.tsx Replaced FaTwitter with FaXTwitter icon and added logic to prevent favicon fallback for X/Twitter domains

Last reviewed commit: 08110d7

<h2>Greptile Overview</h2> <h3>Greptile Summary</h3> Replaced the legacy Twitter bird icon with the modern X logo throughout the referrer display system. The change affects both the icon component and the favicon fallback logic. **Key Changes:** - Switched from `FaTwitter` (from `react-icons/fa`) to `FaXTwitter` (from `react-icons/fa6`) - Updated icon color from `text-blue-400` to neutral colors that adapt to dark mode (`text-neutral-800 dark:text-neutral-200`) - Added `REFERRER_USE_X_ICON` set containing Twitter-related domains (`t.co`, `x.com`, `twitter.com`, `www.twitter.com`) - Modified `getReferrerFavicon` to return `null` for X/Twitter domains, ensuring the custom X icon is always used instead of fetching favicons that might show the old bird logo - Updated changelog to document this as version 0.5.1-alpha <h3>Confidence Score: 5/5</h3> - This PR is safe to merge with no issues - The changes are minimal, well-contained, and correctly implemented. The icon replacement is straightforward with proper dark mode support, and the favicon logic ensures consistent X branding across all Twitter-related domains. The changelog follows project conventions. - No files require special attention <details><summary><h3>Important Files Changed</h3></summary> | Filename | Overview | |----------|----------| | CHANGELOG.md | Added version 0.5.1-alpha entry documenting the X logo icon change with proper changelog formatting and version links | | lib/utils/icons.tsx | Replaced `FaTwitter` with `FaXTwitter` icon and added logic to prevent favicon fallback for X/Twitter domains | </details> </details> <sub>Last reviewed commit: 08110d7</sub>
Sign in to join this conversation.
No description provided.