Skip to content

Feature/navbar header#103

Merged
thaninbew merged 5 commits intomainfrom
feature/navbar-header
Apr 4, 2026
Merged

Feature/navbar header#103
thaninbew merged 5 commits intomainfrom
feature/navbar-header

Conversation

@pujitakalinadhabhotla
Copy link
Copy Markdown
Contributor

Description

Implemented the admin dashboard Sidebar and Header components based on the latest Figma designs. There are a few differences between what is on the screenshot of the ticket and what is on the Figma but I followed the Figma Hi-Fi

Changes Made

  • Frontend changes

  • Created Sidebar component with:
    Navigation items (Dashboard Overview, Donation Tracker, Email Communication, Admin Approval, Settings)

  • Created Header component with:
    Page title ("Dashboard Overview") and User profile section (avatar + name + role)

  • Added temporary tester routes:
    /sidebar-test
    /header-test

Testing & Verification

  • Manual testing completed
  • No breaking changes

Screenshots (if relevant)

Screenshot 2026-03-31 at 2 39 29 PM

Future Improvements/Notes

  • Dynamically change menu item based on which user is logged in

Related Issues

Closes #90

@thaninbew thaninbew self-requested a review April 4, 2026 19:44
Copy link
Copy Markdown
Collaborator

@thaninbew thaninbew 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!

In the interest of time, I made a couple of changes:

  • SVG files were a fine choice for generic icons but because we are already using a component library (react-lucide), I opted with react-lucide icons and removed the SVGs. Note that design uses shadcn components (which are react-lucide anyway..) in Figma.

  • i updated the sidebar to use useLocation from react-router-dom to automatically detect the active route based on the current URL path. it wilkl ensure that the correct navigation item remains highlighted after page refreshes or when deep-linking directly to a specific dashboard page. This is just future proofing for edge cases where we reload or share; the sidebar's highlighting should still be consistent.

  • out of ticket scope but I implemented dynamic username and role in the visible profile.

Image

@thaninbew thaninbew merged commit 140ea18 into main Apr 4, 2026
4 checks passed
@thaninbew thaninbew deleted the feature/navbar-header branch April 4, 2026 20:36
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.

Admin Dashboard Navbar & Header

2 participants