search

fineanmol / Hacktoberfest2024

Make your first Pull Request on Hacktoberfest 2024. Don't forget to spread love and if you like give us a ⭐️
https://fineanmol.github.io/Hacktoberfest2024/
GNU General Public License v3.0
2.39k stars 7.6k forks source link

Enhancement: Navbar should have social media icons only for better UX #7484

Open poswalsameer opened 1 week ago

poswalsameer commented 1 week ago

Is your feature request related to a problem? Please describe. I noticed that the navbar contains text along with social media icons which does not look cool. It should have icons only for a neat and clean UX.

Describe the solution you'd like The navbar should contain only social media icons without any text.

Screenshots Screenshot (64)

fineanmol commented 1 week ago

@poswalsameer Please Star ⭐️ the repo to earn 'hacktober-accepted' label for the event.\nMeanwhile, if you want to work on this issue, please raise a PR, and we will review and merge it.

poswalsameer commented 1 week ago

@poswalsameer Please Star ⭐️ the repo to earn 'hacktober-accepted' label for the event.\nMeanwhile, if you want to work on this issue, please raise a PR, and we will review and merge it.

Hi @fineanmol, already starred and forked. Also created a PR by doing the changes as mentioned in the issue. Looking forward to it getting merged. Thanks in advance!

aditi-agrawal12 commented 1 week ago

Hi @fineanmol If we could add amazing icons using font-awesome and add some hover and transition effects, this would look amazing, please assign this request to me, So that I can make my first contribution in open source 😊

poswalsameer commented 1 week ago

Hi @fineanmol If we could add amazing icons using font-awesome and add some hover and transition effects, this would look amazing, please assign this request to me, So that I can make my first contribution in open source 😊

Hi @aditi-agrawal12, icons are already from Font Awesome, and they have hover effects on them. Still, you can create a new issue and a PR after this my PR gets merged into main.

blizet commented 1 week ago

I have submitted a pull request for this. Could you please assign it to me?