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
This commit is contained in:
Marc-André Bombeck committed 2026-07-29 14:38:19 +02:00
1 parent f3c5ded687
commit b8260474ed
1 file changed
+16 -1
@@ -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}