From bbe75cc1af2f77fc7d00a62a6cef3492c2eeb4bc Mon Sep 17 00:00:00 2001 From: JP Hastings-Spital Date: Wed, 26 Aug 2026 09:57:24 +0100 Subject: [PATCH] fix: drop footer's top border alongside body's padding in empty state (ATFS-dfm3) Collapsing .pf-modal-body's padding when nothing has rendered yet (the previous commit) left the header's bottom border and the footer's own top border sitting flush against each other, reading as one doubled-up line. Same :has() condition, same reasoning: drop the footer's half only in that state. --- ...3--atfsdev-docs-split-into-a-multi-page-diataxis-site.md | 6 +++++- web/src/lib/docs/Search.svelte | 6 ++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/.beans/ATFS-dfm3--atfsdev-docs-split-into-a-multi-page-diataxis-site.md b/.beans/ATFS-dfm3--atfsdev-docs-split-into-a-multi-page-diataxis-site.md index 7aae38a..495865f 100644 --- a/.beans/ATFS-dfm3--atfsdev-docs-split-into-a-multi-page-diataxis-site.md +++ b/.beans/ATFS-dfm3--atfsdev-docs-split-into-a-multi-page-diataxis-site.md @@ -105,7 +105,11 @@ afterthought. selector through `:is(*, #\#)` three times specifically so normal host-page rules can't override it (its own comment says as much); `--pf-*` custom properties are the sanctioned route and never needed - it, but padding/display/margin aren't exposed as one. + it, but padding/display/margin aren't exposed as one. A fourth, + same-cause fix followed: collapsing the body's padding put the + header's bottom border and the footer's own top border flush against + each other, reading as one doubled-up line — dropped the footer's + half in that same empty state. Verified live in a real browser (chrome-devtools MCP): open/close via trigger, shortcut, Escape and backdrop click; a real query with 30 results confirming the header diff --git a/web/src/lib/docs/Search.svelte b/web/src/lib/docs/Search.svelte index 0fb30e2..d66d9ca 100644 --- a/web/src/lib/docs/Search.svelte +++ b/web/src/lib/docs/Search.svelte @@ -190,6 +190,12 @@ :global(pagefind-modal .pf-summary:empty) { margin: 0 !important; } + /* With the body's own padding collapsed above, its zero height puts the + header's bottom border and the footer's top border flush against each + other — reading as one doubled-up line. Drop the footer's half. */ + :global(pagefind-modal .pf-modal-body:has(.pf-summary:empty):has(.pf-results:empty) ~ .pf-modal-footer) { + border-top: none !important; + } /* pagefind-input registers "↓ navigate" unconditionally, whether or not there's anything to navigate to — hide it whenever the results list is -- 2.51.2