chihlasm/resolutionflow
1
0
Fork 0
Code Issues 23 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: Home icon active state — exact match "/" instead of startsWith

Browse Source 
matchPaths: ['/'] with startsWith('/') matched every route,
keeping Home highlighted on all pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
chihlasm
2026-03-22 19:32:55 +00:00
parent 2bcd3e2f3c
commit dfbb51b582
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
frontend/src/components/layout/Sidebar.tsx
View File

@@ -160,7 +160,9 @@ export function Sidebar() {
/* ── Active detection ─────────────────────────────── */ /* ── Active detection ─────────────────────────────── */
const isActive = (item: NavEntry) => { const isActive = (item: NavEntry) => {
if (item.matchPaths) return item.matchPaths.some(p => location.pathname.startsWith(p)) if (item.matchPaths) return item.matchPaths.some(p =>
p === '/' ? location.pathname === '/' : location.pathname.startsWith(p)
)
if (item.href === '/') return location.pathname === '/' if (item.href === '/') return location.pathname === '/'
return location.pathname.startsWith(item.href) return location.pathname.startsWith(item.href)
} }