Update Footer.js

[YadlaMani]

upated the twitter logo

[netlify]

❌ Deploy Preview for codingtrain failed. Why did it fail? →

Name	Link
🔨 Latest commit
🔍 Latest deploy log	https://app.netlify.com/sites/codingtrain/deploys/67028c666e4632538f985fdd

[YadlaMani]

add the hacktoberfest label

[shiffman]

Not sure why this failed, can you elaborate on the icon you've chosen and the changes you propose to make? Glad to consider it, thank you!

[YadlaMani]

@shiffman just added new twitter logo from react-icons in footer

[fturmel]

If @shiffman wants the change, I'd prefer if the icon came from the Font Awesome 6 set like the others, so essentially FaXTwitter from react-icons/fa6.

The icon is also in the about page, not just the footer.

Would you only change the icons and not the labels and keep calling it "Twitter"?

EDIT: The build is failing because the proposed icon was only introduced in react-icons 5 (we're on 4). The substitution I'm recommending is available in v4.

[shiffman]

To be honest, I think I prefer just leaving it as is. However, I do appreciate you @YadlaMani taking the time to engage with this repo! Is there something else you would like to work on? I suggest opening an issue to propose an idea and discuss before submitting a PR. All that said, I would consider @fturmel's suggestion and change the icon (but leave the text as "Twitter."

[YadlaMani]

@shiffman is there any issue i could take upon I am willing to contribute to the repo