Open ashhcoder opened 2 months ago
hey @ashhcoder can you assign this issue to me?
Hey @ashhcoder I would like to work on this issue. Can you please assign this issue to me?
@ashhcoder the twitter logo should be replaced for the main body part too
@ceilican i have pulled a request for the following changes of the twitter logo to "X" logo for the main body and the footer part if you think the pull request is correct kindly merge it as per your convenience.
HI i have changed the twitter logo and replaced it with new one,kindly check my Pull request #PR RAISED
@keshav861 Is Logo changed from your approach ?
@Vegadjay yes logo is changing from my approach
@keshav861 From where you get this logo bcz this is use react-fontawsome library and that library does not contain X logo.. from where you get this logo
@Vegadjay ya i got this the font awesome library doesn't support for the fontawesome V5. react-fontawesome is old library. As of now i was directly using the fontawesome official react component as in node_module i added that component so thats why i think it is working and may be it wont work on it. we have to change the react-fontawesome library to react-icons library this may solve the issue or just add add a javascript file to the html file then can use this directly these are the 2 best ways to overcome this problem i think so....
Hi, I fixed the issue (#265) with the Twitter logo in the body/footer section. the new X logo wasn't available in the old version of FontAwesome, so i updated the FontAwesome packages to the latest version and replaced the logo. It's now looking good. I've submitted a PR with the changes (#267), and i've attached a screenshot for reference. Please review it, and if everything looks good, kindly merge my PR. If there are any issues, please let me know, and I'll make the necessary changes.
Change the twitter bird logo to twitter 'X' logo in the footer section. For reference: