Skip to content

feat: add active navigation state indicator (#383)#471

Merged
Olowodarey merged 1 commit intoArena1X:mainfrom
khanavi272-spec:feat/active-navigation-state-383
Mar 29, 2026
Merged

feat: add active navigation state indicator (#383)#471
Olowodarey merged 1 commit intoArena1X:mainfrom
khanavi272-spec:feat/active-navigation-state-383

Conversation

@khanavi272-spec
Copy link
Copy Markdown
Contributor

🚀 Active Navigation State Indicator (#383)

📋 Overview

Implemented active navigation state so users can easily identify the current page.


✅ Changes Made

  • Integrated usePathname() from Next.js for route detection
  • Added active state styling for navigation links
  • Added visual indicator (underline + color change)
  • Implemented aria-current="page" for accessibility
  • Improved route matching logic for nested routes
  • Applied active state to both desktop and mobile navigation

🧪 Testing

  • Verified active state works correctly on:
    • Home (/)
    • Events (/events)
    • Leaderboard (/leaderboard)
    • Docs (/docs)
    • Dashboard (/dashboard)
  • Tested nested routes (e.g., /docs/xyz)
  • Checked mobile menu active state behavior

♿ Accessibility

  • Added aria-current="page" for screen readers
  • Maintained keyboard navigation and focus behavior

📸 UI Improvements

  • Active link highlighted with:
    • Bold text
    • Underline indicator (desktop)
    • Background highlight (mobile)

Closes #383

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
insight-arena-4rll Ready Ready Preview, Comment Mar 29, 2026 6:22am

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 29, 2026

@khanavi272-spec Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Olowodarey Olowodarey merged commit c3c6da5 into Arena1X:main Mar 29, 2026
3 checks passed
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.

[Frontend] Active Navigation State Indicator

2 participants