From 8b36fd0310a576f10f9dbc819dc121023ab8ecff Mon Sep 17 00:00:00 2001
From: "russell@unturf.com" <russell@unturf.com>
Date: Sun, 17 May 2026 09:14:33 -0400
Subject: [PATCH] =?UTF-8?q?style:=20MPS-24=20=E2=80=94=20drop=20the=20'Fea?=
 =?UTF-8?q?tured'=20rail=20heading=20(it=20lies=20on=20random=20fallback)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Operator: don't label the rail 'Featured' — it falls back to a random
rotation when nothing is curated, so the heading is dishonest there.

- Removed the <h2>Featured</h2> from the featured_rail macro; cards
  stand on their own. aria-label 'Featured products' -> neutral
  'More from this shop' (accurate in both curated + random states).
- Removed the now-dead h2.serp-rail-title CSS rule + styleguide
  heading.
- Test asserts the heading is gone (serp-rail-title / >Featured<
  absent) while the rail itself still renders. 1139 passed.

CSS/templates cache-busted.
---
 make_post_sell/static/css/common.css    | 7 -------
 make_post_sell/templates/_facet_nav.j2  | 6 ++++--
 make_post_sell/templates/styleguide.j2  | 3 +--
 make_post_sell/tests/test_functional.py | 4 +++-
 4 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/make_post_sell/static/css/common.css b/make_post_sell/static/css/common.css
index 5e6b3ea..23a25dd 100644
--- a/make_post_sell/static/css/common.css
+++ b/make_post_sell/static/css/common.css
@@ -1976,13 +1976,6 @@ aside.serp-rail {
     gap: var(--space-3, 12px);
     align-content: start;
 }
-h2.serp-rail-title {
-    margin: 0;
-    font-size: var(--type-body-size, 1rem);
-    font-weight: 600;
-    color: var(--text-color, #333);
-    letter-spacing: 0.02em;
-}
 ul.serp-rail-list {
     list-style: none;
     margin: 0;
diff --git a/make_post_sell/templates/_facet_nav.j2 b/make_post_sell/templates/_facet_nav.j2
index 154fd22..c3187fd 100644
--- a/make_post_sell/templates/_facet_nav.j2
+++ b/make_post_sell/templates/_facet_nav.j2
@@ -145,8 +145,10 @@
    inside .tag-detail-layout. #}
 {% macro featured_rail(products, request) %}
 {% if products %}
-<aside class="serp-rail" aria-label="Featured products">
-  <h2 class="serp-rail-title type-title">Featured</h2>
+<aside class="serp-rail" aria-label="More from this shop">
+  {# No "Featured" heading: the rail falls back to a random rotation
+     when nothing is curated, so labelling it "Featured" would lie.
+     The cards stand on their own. #}
   <ul class="serp-rail-list">
   {% for p in products %}
     <li class="serp-rail-card">
diff --git a/make_post_sell/templates/styleguide.j2 b/make_post_sell/templates/styleguide.j2
index f463151..9889afb 100644
--- a/make_post_sell/templates/styleguide.j2
+++ b/make_post_sell/templates/styleguide.j2
@@ -1193,8 +1193,7 @@ The snippet sets inline margin-left for depth-N replies (depth * 20px).</div>
       a sticky right column &ge;1100px. Grid only.
     </p>
     <div class="sg-demo">
-      <aside class="serp-rail" aria-label="Featured products" style="max-width:280px;">
-        <h2 class="serp-rail-title type-title">Featured</h2>
+      <aside class="serp-rail" aria-label="More from this shop" style="max-width:280px;">
         <ul class="serp-rail-list">
           <li class="serp-rail-card">
             <span class="serp-rail-thumb-link"><span class="serp-rail-thumb" style="display:block;aspect-ratio:1;background:var(--surface-dim,#f3f4f6);border-radius:var(--radius-sm,4px);"></span></span>
diff --git a/make_post_sell/tests/test_functional.py b/make_post_sell/tests/test_functional.py
index cf70744..5a6309b 100644
--- a/make_post_sell/tests/test_functional.py
+++ b/make_post_sell/tests/test_functional.py
@@ -9022,7 +9022,9 @@ class TestHomeLayoutAndTags(_AuthenticatedBase):
         body = res.body.decode()
         self.assertIn('class="serp-rail"', body)
         self.assertIn('serp-rail-list', body)
-        self.assertIn('>Featured<', body)
+        # No "Featured" heading — it would lie on the random fallback.
+        self.assertNotIn('serp-rail-title', body)
+        self.assertNotIn('>Featured<', body)
         rail = body.split('class="serp-rail"', 1)[1]
         self.assertIn(product.title, rail)