Skip to content

642/toolbar-touchup#645

Open
samikabhatkar-web wants to merge 7 commits intodevfrom
642/toolbar-touchup
Open

642/toolbar-touchup#645
samikabhatkar-web wants to merge 7 commits intodevfrom
642/toolbar-touchup

Conversation

@samikabhatkar-web
Copy link
Contributor

…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
Screenshot 2026-02-21 at 4 47 53 PM Screenshot 2026-02-21 at 4 47 46 PM

Copy link
Collaborator

@LexTruong LexTruong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@samikabhatkar-web samikabhatkar-web changed the title feat(frontend): ✨ added conditional toolbar styling based on current … 642/toolbar-touchup Feb 26, 2026
Copy link
Collaborator

@LexTruong LexTruong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Contributor Author

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants