Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

88 update navbar #95

Merged
merged 6 commits into from
Mar 11, 2025
Merged

88 update navbar #95

merged 6 commits into from
Mar 11, 2025

Conversation

kaitlinnleung
Copy link
Contributor

Description

Update Navbar to match Figma, verify Navbar paths

Screenshots/Media

Issues

Closes #88

@theNatePi
Copy link
Collaborator

Good work @kaitlinnleung @Xire7! Navbar looks great. I just have a small change to make

  • Clicking on the programs (calendar) button should navigate to /programs and show the program dashboard page (still called Home). Make sure that /programs routes to the Home component, instead of a single program component

theNatePi
theNatePi requested changes Mar 5, 2025
View reviewed changes
Copy link
Collaborator

@theNatePi theNatePi left a comment

Choose a reason for hiding this comment

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

Fix needed

@theNatePi
Copy link
Collaborator

Looks good to me!

@theNatePi theNatePi merged commit 9acc2cc into main Mar 11, 2025
1 check passed
@theNatePi theNatePi deleted the 88-update-navbar branch March 11, 2025 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theNatePi theNatePi theNatePi requested changes

@jessieh9 jessieh9 Awaiting requested review from jessieh9

Assignees
No one assigned
Labels
Fix Needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Navbar
3 participants
@kaitlinnleung @theNatePi @Xire7