From 165f2d38feb6cac9732e7c4220b4b968039d6379 Mon Sep 17 00:00:00 2001
From: Mao Nakamoto <41178744+maonakamoto@users.noreply.github.com>
Date: Mon, 31 Aug 2026 13:46:40 +0200
Subject: [PATCH] fix(nav): the current page was coloured, never announced

`isActive` already decided the link's colour and set no aria-current, so the
current page existed only for people who can see it. Zero occurrences of
aria-current anywhere in this codebase.

The megamenu trigger is untouched: it is a disclosure button, not a link to a
page, and Popover.Button already announces aria-expanded/aria-haspopup itself.

No visual change.

Found by the fleet nav-contract audit, which flags a repo whose navigation
computes an active state and never announces it anywhere.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XeELB8b3N4JrT2asYL9WvE
---
 components/navigation/NavItem.tsx | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/components/navigation/NavItem.tsx b/components/navigation/NavItem.tsx
index e6c73e00..55229428 100644
--- a/components/navigation/NavItem.tsx
+++ b/components/navigation/NavItem.tsx
@@ -47,6 +47,9 @@ export function NavItem({ item, isActive, onNavigate }: NavItemProps) {
     return (
       <Link
         href={item.path}
+        // `isActive` already decides the colour; it never told assistive tech
+        // which page you were on. Zero aria-current existed in this codebase.
+        aria-current={isActive ? 'page' : undefined}
         className={`text-sm font-medium transition-colors ${
           isActive ? 'text-action' : 'text-gray-600'
         } hover:text-action`}