642/toolbar-touchup

[samikabhatkar-web]

…route

Summary

• Revised the toolbar to have conditional styling based on the current page, matching Figma styles
• As per the figma, the restaurant pages (Brandywine, Anteatery) and the About Us page have the original transparent look with white text. All other pages use a white background with the blue PeterPlate title and black nav links.
• Should be dark mode compatible

Changes

• Added conditional styling to Toolbar based on current route using usePathname

[LexTruong]

Great work! Just a couple of minor fixes needed to further align with the Figma:

• When users are not logged in, there should only be a sign in button, not a menu icon as well.
• The menu button is not visible (white on white) on the about page when in light mode.

[LexTruong]

Hey Samika! I helped merge your changes with dev to resolve the conflicts. This merge brought a new MobileToolbar in toolbar.tsx. Please make the top bar of the mobile toolbar transparent, similar to how you did for the desktop toolbar. Also, please add the thin grey line at the bottom of the the desktop toolbar on non-transparent pages.

[samikabhatkar-web]

Hey Samika! I helped merge your changes with dev to resolve the conflicts. This merge brought a new MobileToolbar in toolbar.tsx. Please make the top bar of the mobile toolbar transparent, similar to how you did for the desktop toolbar. Also, please add the thin grey line at the bottom of the the desktop toolbar on non-transparent pages.

Got it, just addressed those changes!

[LexTruong]

Hey Samika, those changes you made look great! 2 other things I noticed though (sorry if I missed them the first time):

• Missing a link to the Meal tracker page in the toolbar
• Let's disable the Food Courts button for now since it's not implemented

[TheAbubakrAbu]

Hi Samika! Just following up on the previous comments from Lex:

• The Meal Tracker link still seems to be missing from the toolbar
• The Food Courts button is still enabled even though it’s not implemented yet

Just wanted to check if this is still being worked on. Let us know if you need anything!

[samikabhatkar-web]

Hi Samika! Just following up on the previous comments from Lex:

• The Meal Tracker link still seems to be missing from the toolbar
• The Food Courts button is still enabled even though it’s not implemented yet

Just wanted to check if this is still being worked on. Let us know if you need anything!

Hi Abu, I'll get to these changes today!

[TheAbubakrAbu]

Looks good to me, thanks for the fixes!