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.

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

Update Navbar #88

Closed
4 tasks
jessieh9 opened this issue Feb 22, 2025 · 0 comments · Fixed by #95
Closed
4 tasks

Update Navbar #88

jessieh9 opened this issue Feb 22, 2025 · 0 comments · Fixed by #95
Assignees

Comments

@jessieh9
Copy link
Collaborator

The navbar has been updated to be simpler and smaller on the left side of each page. Update the navbar to match the Figma design. In addition to this, update the links for the /home page and the state of the navbar when clicking on pages.

  1. Update Navbar to match Figma
    Update the Navbar in client/src/components/navbar/Navbar.jsx to match the Figma
    Mind: the responsiveness of the navbar. It should allow a page to be nested inside, and should allow for dynamic sizing

The navbar should maintain its state and be able to detect which page it is on to properly render which page on the right is "selected"
For example, if I navigate to /invoices, the navbar should update to show that I am on an invoices page.
Also, the icon should stay selected for related pages.
/programs includes /programs/*
/invoices influences /invoices/*
Where the * captures any link or id following

  1. Make sure links on the navbar work properly
    The calendar should link to /programs
    The invoice icon should link to /invoices
    The notifications icon should link to /notifications
    The settings icon should link to /settings

  2. Change routing of /home to /programs
    Previously, we had the programs page on /home. Change this route in App.jsx to /programs. Also make sure to update all pages and buttons which link to /home to /programs

Acceptance Criteria

  • The Navbar component is updated to match the Figma
    • All links are connected
    • The buttons have proper hover and select states
  • The Nabar works on all pages and allows for dynamic nesting of pages inside
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 a pull request may close this issue.

3 participants