Skip to content
This repository has been archived by the owner on Sep 7, 2024. It is now read-only.

Twitter Icon Updated to X using react-icons (not using any svg) #797

Merged

Conversation

maverick-farhan
Copy link
Contributor

@maverick-farhan maverick-farhan commented Sep 30, 2023

Fixes Issue

This PR fixes the following issues:

closes #772

Changes proposed

Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

  • My code follows the code style of this project.
  • The title of my pull request is a short description of the requested changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.

Screenshots

twiiter-icon-changed-to-X
screenshot

code-screenshot

Add all the screenshots which support your changes

Refactor: Updated the Twitter icon in the Footer component. The previously used FaTwitter icon from react-icons/fa has been replaced with FaXTwitter icon from react-social-icon package.

Summary by CodeRabbit

  • Style: Updated the Twitter icon in the footer to use FaXTwitter instead of FaTwitter.
  • Style: Changed the hover color for the Twitter link in the footer to #FAF3F0 from #1DA1F2 for better visual consistency.

@coderabbitai
Copy link

coderabbitai bot commented Sep 30, 2023

Walkthrough

The proposed changes primarily revolve around the visual aspects of the Footer component. The Twitter icon has been replaced, and the hover color for the Twitter link has been updated to a new shade.

Changes

File Summary
src/components/layout/Footer.jsx The Twitter icon (FaTwitter) is replaced with a new one (FaXTwitter). The hover color for the Twitter link has also been updated from #1DA1F2 to #FAF3F0.

🐇💻

In the land of code where the shadows lie,

An icon changes under the digital sky.

A new hue on hover, a subtle delight,

Enhancing user experience, making it just right.

So here's to the change, may it be ever so bright! 🎉🌟


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • Mention @coderabbitai in any review comment for bot assistance.
  • Note: Review comments are made on code diffs or files, not on the PR overview.

Pause Incremental Reviews

  • Insert @coderabbitai: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.
  • Additionally, enter @coderabbitai pause to halt the bot's ongoing reviews as any issue or review comment. To
    resume reviews, enter @coderabbitai resume as any issue or review comment.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 0

Commits Files that changed from the base of the PR and between 2578677 and 0334845.
Files ignored due to filter (2)
  • package-lock.json
  • package.json
Files selected for processing (1)
  • src/components/layout/Footer.jsx (2 hunks)
Files skipped from review due to trivial changes (1)
  • src/components/layout/Footer.jsx

Copy link
Contributor Author

@maverick-farhan maverick-farhan left a comment

Choose a reason for hiding this comment

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

Updated twitter icon to X would you please review my changes. <3

@maverick-farhan maverick-farhan changed the title Twitter Icon Updated to X using react-icon Twitter Icon Updated to X using react-icons (not using any svg) Oct 2, 2023
@AnuragThePathak AnuragThePathak merged commit 7f1b656 into WeMakeDevs:main Oct 10, 2023
4 checks passed
This was referenced Oct 10, 2023
@maverick-farhan maverick-farhan deleted the twitter-icon-changed-to-X branch October 10, 2023 13:16
@maverick-farhan maverick-farhan restored the twitter-icon-changed-to-X branch October 10, 2023 13:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request to Update Twitter Icon to X
2 participants