fix: sidebar highlighting works on all tabs, not just exchanges
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>
This commit is contained in:
@@ -323,11 +323,9 @@ export function Sidebar({ apps, className, onNavigate }: SidebarProps) {
|
|||||||
const starredRouteStats = starredItems.filter((i) => i.type === 'routestat')
|
const starredRouteStats = starredItems.filter((i) => i.type === 'routestat')
|
||||||
const hasStarred = starredItems.length > 0
|
const hasStarred = starredItems.length > 0
|
||||||
|
|
||||||
// For exchange detail pages, use the reveal path for sidebar selection so
|
// When a sidebar reveal path is provided (e.g. via Cmd-K navigation),
|
||||||
// the parent route is highlighted (exchanges have no sidebar entry of their own)
|
// use it for sidebar selection so the correct item is highlighted
|
||||||
const effectiveSelectedPath = location.pathname.startsWith('/exchanges/') && sidebarRevealPath
|
const effectiveSelectedPath = sidebarRevealPath ?? location.pathname
|
||||||
? sidebarRevealPath
|
|
||||||
: location.pathname
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<aside className={`${styles.sidebar} ${className ?? ''}`}>
|
<aside className={`${styles.sidebar} ${className ?? ''}`}>
|
||||||
|
|||||||
Reference in New Issue
Block a user