search

VanshKing30 / FoodiesWeb

GssoC'24 Project | Open To Contributers
https://foodies-web-app.vercel.app
MIT License
61 stars 155 forks source link

Responsive Navbar #100

Closed The-Phoenics closed 6 months ago

The-Phoenics commented 6 months ago

Hi @VanshKing30, I've noticed following problems with navbar:

  1. Currently there is no navbar styling to indicate the current page user is navigating.
  2. Navbar is not responsive on smaller screen sizes.
  3. Nav elements text(not centered) styling is quite not desirable: Screenshot_2024_0512_230852.jpg

I would like to work on this for GSSoC'23. Can you please assign this to me?

VanshKing30 commented 6 months ago

What changes can you make , please share some screenshots

The-Phoenics commented 6 months ago

Hey @VanshKing30 , these are some issues which I could work and fix in navbar:

Hover color on login button doesn't look as good in my opinion: 3 1

Text in nav elements is not center aligned: 2

Extra padding on left side of navbar on small size screen. Either it could be removed or some padding could be added on right side! 4

And also a small pop-like framer motion animation would look good on nav elements, and some highlight inidicating which page is active.

VanshKing30 commented 6 months ago

@The-Phoenics How long will it take you to solve this issue ?

The-Phoenics commented 6 months ago

@The-Phoenics How long will it take you to solve this issue ?

I can do these fixes by tomorrow.