From b8260474ed8374493ee418123165dcd3c0890d6b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <mbombeck@gmail.com>
Date: Wed, 29 Jul 2026 14:38:19 +0200
Subject: [PATCH] fix(01-17): remove duplicate mobile insights spacer

- suppress only the inherited mobile pre-pill top padding
- preserve safe-area clearance focus paint and 44px targets
---
 .../insights/insights-layout-shell.tsx          | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/components/insights/insights-layout-shell.tsx b/src/components/insights/insights-layout-shell.tsx
index 5e7d0dff0..bc861d344 100644
--- a/src/components/insights/insights-layout-shell.tsx
+++ b/src/components/insights/insights-layout-shell.tsx
@@ -180,7 +180,22 @@ export function InsightsLayoutShell({ children }: { children: ReactNode }) {
   }, [layoutSettled, insightsLayout]);
 
   return (
-    <div className="space-y-8">
+    <div data-slot="insights-layout-shell" className="space-y-8">
+      {/*
+        AuthShell gives every route a generic `pt-6`. Insights already owns a
+        sticky, padded navigation row as its first surface, so that generic
+        spacer became a second unexplained 24px gap on phones. Remove only
+        that inherited top padding when this shell is the direct route child.
+        Horizontal padding and safe-area-aware bottom clearance stay intact;
+        no negative margin can clip pill focus.
+      */}
+      <style jsx global>{`
+        @media (max-width: 767px) {
+          #main-content > div:has(> [data-slot="insights-layout-shell"]) {
+            padding-top: 0;
+          }
+        }
+      `}</style>
       <InsightsTabStrip
         onRegenerate={advisorEnabled ? advisor.regenerate : undefined}
         regenerating={advisor.isRegenerating}