Skip to content

add active link indicator#812

Open
Khushi281300 wants to merge 1 commit intoOWASP:mainfrom
Khushi281300:feat/navbar-active-647
Open

add active link indicator#812
Khushi281300 wants to merge 1 commit intoOWASP:mainfrom
Khushi281300:feat/navbar-active-647

Conversation

@Khushi281300
Copy link

@Khushi281300 Khushi281300 commented Mar 15, 2026

Summary

This PR resolves #647 by adding a clear and reliable active-state indicator to the main navbar.
It improves navigation clarity across the application while maintaining the project's premium design aesthetic.


Changes Included

  • Active Route Detection
    Implemented useLocation and a custom isActive helper in Header.tsx for accurate route matching, including sub-pages.

  • Clean Styling
    Added nav-link--active styles in header.scss with:

    • font-weight: 600
    • white text
    • blue underline indicator
  • UX Improvements
    Added smooth transitions for the underline highlight.

  • Mobile Support
    Verified and styled active states in the mobile navigation menu.

  • Maintainability
    Only the two relevant files were modified. No changes were made to yarn.lock or project dependencies.


Why This Is Needed

Currently, users cannot easily identify which page is active in the navbar.
This update provides a clear visual indicator, improving navigation usability and aligning with the application's clean, professional design.


Visual Proof

result2 result4

@Khushi281300 Khushi281300 changed the title add active link indicator (underline + white highlight) (#647) add active link indicator Mar 15, 2026
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.

1 participant