From 8175df780321107d4f2b481f4720ff802fdfae7a Mon Sep 17 00:00:00 2001
From: Tanmay Chaturvedi <168345467+tanmayc43@users.noreply.github.com>
Date: Sun, 8 Dec 2024 20:16:39 +0530
Subject: [PATCH] Chore - Updated twitter.svg (#4871)

Updated the old Twitter bird logo to the new "X" logo.

## Because
The old twitter bird logo has been long replaced by the new "X" logo and
I have done the same on the website to keep it updated and improve the
overall user experience.

## This PR
-The old bird logo SVG file was replaced by the new "X" logo SVG file.
-The fill rules were modified accordingly.



<!--
If this PR closes an open issue in this repo, replace the XXXXX below
with the issue number, e.g. Closes #2013.

If this PR closes an open issue in another TOP repo, replace the #XXXXX
with the URL of the issue, e.g. Closes
https://github.com/TheOdinProject/curriculum/issues/XXXXX

If this PR does not close, but is related to another issue or PR, you
can link it as above without the 'Closes' keyword, e.g. 'Related to
#2013'.
-->


<!-- Any other information about this PR, such as a link to a Discord
discussion. -->


## Pull Request Requirements
<!-- Replace the whitespace between the square brackets with an 'x',
e.g. [x]. After you create the PR, they will become checkboxes that you
can click on. -->
- [x] I have thoroughly read and understand [The Odin Project
Contributing
Guide](https://github.com/TheOdinProject/theodinproject/blob/main/CONTRIBUTING.md)
- [x] The title of this PR follows the `keyword: brief description of
change` format, using one of the following keywords:
    - `Feature` - adds new or amends existing user-facing behavior
- `Chore` - changes that have no user-facing value, refactors,
dependency bumps, etc
    - `Fix` - bug fixes
-   [x] The `Because` section summarizes the reason for this PR
- [x] The `This PR` section has a bullet point list describing the
changes in this PR
- [x] I have verified all tests and linters pass after making these
changes.
- [x] If this PR addresses an open issue, it is linked in the `Issue`
section
-   [x] If applicable, this PR includes new or updated automated tests
---
 app/assets/images/socials/twitter.svg | 7 +++++--
 app/views/shared/_footer.html.erb     | 3 ++-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/assets/images/socials/twitter.svg b/app/assets/images/socials/twitter.svg
index f1217433db..747ee1234f 100644
--- a/app/assets/images/socials/twitter.svg
+++ b/app/assets/images/socials/twitter.svg
@@ -1,3 +1,6 @@
-<svg fill="currentColor" viewBox="0 0 24 24" aria-hidden="true">
-  <path d="M8.29 20.251c7.547 0 11.675-6.253 11.675-11.675 0-.178 0-.355-.012-.53A8.348 8.348 0 0022 5.92a8.19 8.19 0 01-2.357.646 4.118 4.118 0 001.804-2.27 8.224 8.224 0 01-2.605.996 4.107 4.107 0 00-6.993 3.743 11.65 11.65 0 01-8.457-4.287 4.106 4.106 0 001.27 5.477A4.072 4.072 0 012.8 9.713v.052a4.105 4.105 0 003.292 4.022 4.095 4.095 0 01-1.853.07 4.108 4.108 0 003.834 2.85A8.233 8.233 0 012 18.407a11.616 11.616 0 006.29 1.84" />
+<svg role="img" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg">
+  <title>X</title>
+  <g transform="scale(0.85) translate(1.8, 1.8)">
+    <path d="M18.901 1.153h3.68l-8.04 9.19L24 22.846h-7.406l-5.8-7.584-6.638 7.584H.474l8.6-9.83L0 1.154h7.594l5.243 6.932ZM17.61 20.644h2.039L6.486 3.24H4.298Z"/>
+  </g>
 </svg>
diff --git a/app/views/shared/_footer.html.erb b/app/views/shared/_footer.html.erb
index 93356496ae..78f7954ba9 100644
--- a/app/views/shared/_footer.html.erb
+++ b/app/views/shared/_footer.html.erb
@@ -47,8 +47,9 @@
 
           <%= link_to 'https://twitter.com/TheOdinProject', class: 'text-gray-400 hover:text-gray-500 dark:text-gray-500 dark:hover:text-gray-400' do %>
             <span class="sr-only">Twitter</span>
-            <%= inline_svg_tag 'socials/twitter.svg', class: 'h-6 w-6 fill-current hover:fill-[#1DA1F2] transition duration-500', aria: true, title: 'Twitter', desc: 'Twitter logo' %>
+            <%= inline_svg_tag 'socials/twitter.svg', class: 'h-6 w-6 fill-current hover:fill-black dark:hover:fill-white transition duration-500', aria: true, title: 'Twitter', desc: 'Twitter logo' %>
           <% end %>
+
         </div>
       </div>