Skip to content

style: add active navigation state indicator#996

Merged
komalharshita merged 2 commits into
komalharshita:mainfrom
bhatiakavisha:style/active-navbar-indicator
Jul 5, 2026
Merged

style: add active navigation state indicator#996
komalharshita merged 2 commits into
komalharshita:mainfrom
bhatiakavisha:style/active-navbar-indicator

Conversation

@bhatiakavisha

Copy link
Copy Markdown
Contributor

Description

Implemented dynamic active navigation state indicators across the application navbar using Jinja request.path checks.

Changes Made

  • Replaced hardcoded active navigation styling in compare.html.
  • Added dynamic active state support for navigation links in contact.html.
  • Added active state support for mobile navigation links in index.html.
  • Reused the existing .active styles and underline animations already defined in the project.
  • Ensured active indicators update automatically based on the current route.

Testing

  • Verified active navigation states on:

    • Home page
    • Compare page
    • Contact page
  • Tested desktop and mobile navigation menus.

  • Ran test suite successfully.

Issue

Closes #751

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

@bhatiakavisha is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@komalharshita komalharshita merged commit 817d256 into komalharshita:main Jul 5, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Active Navigation State Indicator in Navbar

2 participants