From 98c3df860b1af7d78b129ae3248d5293f1f83692 Mon Sep 17 00:00:00 2001
From: giles <giles.bradshaw@sigyl.com>
Date: Thu, 19 Feb 2026 15:28:51 +0000
Subject: [PATCH] Fix category selector highlighting to use slug comparison

Use top_slug/sub_slug directly instead of current_local_href for
active state detection. The previous approach compared full request
paths against short category-relative paths, which never matched.
This also avoids conflicting with current_local_href used by brand
filter URL construction.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../templates/_types/browse/desktop/_category_selector.html   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/browser/templates/_types/browse/desktop/_category_selector.html b/browser/templates/_types/browse/desktop/_category_selector.html
index ba642b7..b3c68b6 100644
--- a/browser/templates/_types/browse/desktop/_category_selector.html
+++ b/browser/templates/_types/browse/desktop/_category_selector.html
@@ -2,7 +2,7 @@
 <nav aria-label="Categories"
      class="rounded-xl border bg-white shadow-sm min-h-0">
   <ul class="divide-y">
-    {% set top_active = (current_local_href == top_local_href) %}
+    {% set top_active = (sub_slug is not defined or sub_slug is none or sub_slug == '') %}
     {% set href = (url_for('market.browse.browse_top', top_slug=top_slug) ~ qs)|host %}
     <li>
       <a
@@ -19,7 +19,7 @@
     </li>
 
     {% for sub in subs_local %}
-      {% set active = (current_local_href == sub.local_href) %}
+      {% set active = (sub.slug == sub_slug) %}
       {% set href = (url_for('market.browse.browse_sub', top_slug=top_slug, sub_slug=sub.slug) ~ qs)|host %}
       <li>
         <a