From 03d3620b02d4a830e3243bcd920315e4c1d809c3 Mon Sep 17 00:00:00 2001 From: Luke Melia Date: Thu, 4 Jun 2026 17:41:34 -0400 Subject: [PATCH 1/3] Nudge the below-card Adorn label down 1px so it doesn't ride high MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When the type-label tab flips below a card (no room above), it overlapped the card's bottom edge by 2px, leaving it sitting one pixel too high over the edge. Reduce the overlap to 1px so the flag's top edge still tucks under the bottom outline stroke and reads as attached. Addresses the vertical-alignment point of CS-11331; verified live (overlap 2px → 1px). Co-Authored-By: Claude Opus 4.8 (1M context) --- packages/host/app/modifiers/position-adorn-label.ts | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/packages/host/app/modifiers/position-adorn-label.ts b/packages/host/app/modifiers/position-adorn-label.ts index 2b56b91694a..bac70a695f5 100644 --- a/packages/host/app/modifiers/position-adorn-label.ts +++ b/packages/host/app/modifiers/position-adorn-label.ts @@ -116,12 +116,13 @@ export function makePositionAdornLabel( anchorLeftX = cardRect.left - 4; label.style.maxWidth = 'max-content'; } - // When flipped below, overlap the card by 2px so the flag's wide - // top edge tucks under the bottom outline stroke and reads as - // attached. (Anchoring it 2px below the card instead left a - // visible gap under the stroke.) + // When flipped below, overlap the card by 1px so the flag's wide + // top edge tucks just under the bottom outline stroke and reads as + // attached without sitting too high over the card edge. (A 2px + // overlap rode one pixel too high; anchoring it below the card + // instead left a visible gap under the stroke.) let anchorTopY = - side === 'top' ? cardRect.top - labelHeight - 2 : cardRect.bottom - 2; + side === 'top' ? cardRect.top - labelHeight - 2 : cardRect.bottom - 1; // The label's anchor positions (anchorLeftX, anchorTopY) are in // viewport coordinates. With `position: absolute`, the inline From 7d7b8806be401e895c5f4ee080a7567e6fc901aa Mon Sep 17 00:00:00 2001 From: Luke Melia Date: Thu, 4 Jun 2026 18:32:29 -0400 Subject: [PATCH 2/3] Remove the grey drop-shadow peek on the Adorn label tab (CS-11331) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Adorn label tab cast a grey `drop-shadow` filter that bled out past its clipped/rounded corners as a grey peek. Remove it. Combined with the 1px below-card nudge, the tab reads as cleanly attached to the card. Two of the three CS-11331 points are addressed here (grey peek and the below-card vertical alignment). The border-radius point is left as a deliberate design call — the tab is a separate 24px-tall flag and the card is 10px, so force-matching the radii would make the small tab look disproportionate — and CS-11331 stays open for it. Co-Authored-By: Claude Opus 4.8 (1M context) --- packages/host/app/components/adorn/adorn-label.gts | 1 - 1 file changed, 1 deletion(-) diff --git a/packages/host/app/components/adorn/adorn-label.gts b/packages/host/app/components/adorn/adorn-label.gts index 225ce96a853..7bfb3065ef3 100644 --- a/packages/host/app/components/adorn/adorn-label.gts +++ b/packages/host/app/components/adorn/adorn-label.gts @@ -69,7 +69,6 @@ const AdornLabel: TemplateOnlyComponent =