From 1181729da59ad0596db5bffc2ebf6e732f64227a Mon Sep 17 00:00:00 2001 From: Sam Chau Date: Sun, 9 Nov 2025 07:11:12 +1100 Subject: [PATCH] fix: update active state logic to include nested routes in navigation components --- src/lib/client/ui/navigation/pageNav/groupHeader.svelte | 2 +- src/lib/client/ui/navigation/pageNav/groupItem.svelte | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lib/client/ui/navigation/pageNav/groupHeader.svelte b/src/lib/client/ui/navigation/pageNav/groupHeader.svelte index 6f1ea8e..a70518b 100644 --- a/src/lib/client/ui/navigation/pageNav/groupHeader.svelte +++ b/src/lib/client/ui/navigation/pageNav/groupHeader.svelte @@ -11,7 +11,7 @@ let { label, href, isOpen, hasItems, onToggle }: Props = $props(); - const isActive = $derived($page.url.pathname === href); + const isActive = $derived($page.url.pathname === href || $page.url.pathname.startsWith(href + '/'));
diff --git a/src/lib/client/ui/navigation/pageNav/groupItem.svelte b/src/lib/client/ui/navigation/pageNav/groupItem.svelte index 989a5b7..38f223a 100644 --- a/src/lib/client/ui/navigation/pageNav/groupItem.svelte +++ b/src/lib/client/ui/navigation/pageNav/groupItem.svelte @@ -8,7 +8,7 @@ let { label, href }: Props = $props(); - const isActive = $derived($page.url.pathname === href); + const isActive = $derived($page.url.pathname === href || $page.url.pathname.startsWith(href + '/'));