From 48a5eb715dc03ed7a2ae9f481f783b8b6e9487f5 Mon Sep 17 00:00:00 2001 From: "russell@unturf.com" Date: Sat, 16 May 2026 05:33:58 -0400 Subject: [PATCH] fix: link tag chip strip to the facet left-nav (same targets) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The chip strip linked to ?tag= (in-place filtered shop home) while the facet sidebar/details Categories list linked to {tag_base}/tag/ (the canonical tag detail page). Clicking the same category in the two navs took you to two different pages with independently-computed active states — they were never in sync. Point the chip strip at the same targets the facet nav uses: - "All" -> shop home ({{ shop_url }}) (unchanged) - category -> {{ tag_base }}/tag/{{ slug }} (was ?tag={{ slug }}) Lane "See all ->" links moved the same way for consistency. tag_filter.js already keeps full navigation for /tag/ hrefs (it bails on init when the chip href contains /tag/), so this needs no JS change — chips and the left nav now land on the identical page with the identical active highlight. Server still resolves active_tag from both /tag/ and any legacy ?tag= param, so old links keep working. 33 tests in the home-layout / chip / facet / tag-detail / lane slice pass. --- make_post_sell/templates/home.j2 | 9 ++++++--- make_post_sell/templates/shop.j2 | 10 ++++++++-- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/make_post_sell/templates/home.j2 b/make_post_sell/templates/home.j2 index 94c14bd..5dfee8e 100644 --- a/make_post_sell/templates/home.j2 +++ b/make_post_sell/templates/home.j2 @@ -88,7 +88,10 @@ Layout 0 (flat) keeps its existing simple grid with no nav. #} {% set show_facet_nav = request.shop.home_layout and request.shop.home_layout >= 1 %} - {# MPS-24: tag chip strip — shown on layout 1 (chips) and 2 (lanes). #} + {# MPS-24: tag chip strip — shown on layout 1 (chips) and 2 (lanes). + Chips and the facet left-nav point at the SAME targets (shop home + for "All", {tag_base}/tag/{slug} per category) so the two stay in + lock-step. tag_filter.js keeps full navigation for /tag/ hrefs. #} {% if home_chips %}