Compare commits

...

1 Commits

Author SHA1 Message Date
hsiegeln
af48bd2fa0 fix: sidebar highlighting works on all tabs, not just exchanges
All checks were successful
Build & Publish / publish (push) Successful in 1m0s
SonarQube Analysis / sonarqube (push) Successful in 2m17s
Remove the /exchanges/ path guard so sidebarRevealPath is used whenever
available, enabling correct sidebar selection on dashboard, runtime, and
all other tabs when navigating via Cmd-K.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-01 20:59:15 +02:00

View File

@@ -323,11 +323,9 @@ export function Sidebar({ apps, className, onNavigate }: SidebarProps) {
const starredRouteStats = starredItems.filter((i) => i.type === 'routestat')
const hasStarred = starredItems.length > 0
// For exchange detail pages, use the reveal path for sidebar selection so
// the parent route is highlighted (exchanges have no sidebar entry of their own)
const effectiveSelectedPath = location.pathname.startsWith('/exchanges/') && sidebarRevealPath
? sidebarRevealPath
: location.pathname
// When a sidebar reveal path is provided (e.g. via Cmd-K navigation),
// use it for sidebar selection so the correct item is highlighted
const effectiveSelectedPath = sidebarRevealPath ?? location.pathname
return (
<aside className={`${styles.sidebar} ${className ?? ''}`}>