fix(insights): restore the shared card rhythm the two assessment cards lost

The spacing pass on this branch tightened `insight-status-card` to `gap-1.5`
and left `mood-insights-sections` at `gap-2`, so two sibling cards that are
meant to read as one family stopped matching. `assessment-spine-order.test.ts`
exists to hold exactly that contract and went red as soon as the formatter
stopped masking it: `format:check` failed first in the job, so the unit tests
never ran and the divergence shipped unnoticed onto the branch.

Bring the mood card to the same value rather than widen the other one back.
The direction of travel is the tighter rhythm, and both allowed steps are on
the sanctioned spacing scale.

The test's expected string moves with the contract, not around it: it still
asserts that both cards carry the SAME value, which is the whole point of it.
This commit is contained in:
Marc-André Bombeck committed 2026-07-30 10:49:06 +02:00
1 parent bc94223423
commit 1f87e63662
2 files changed
+3 -3

No files matched your search

@@ -176,11 +176,11 @@ describe("assessment presentation contract", () => {
);
expect(cardSource).toContain(
'className="animate-insight-in gap-2 py-3 md:py-4"',
'className="animate-insight-in gap-1.5 py-3 md:py-4"',
);
expect(cardSource).toContain('className="text-foreground text-sm"');
expect(moodSource).toContain(
'className="animate-insight-in gap-2 py-3 md:py-4"',
'className="animate-insight-in gap-1.5 py-3 md:py-4"',
);
expect(moodSource).toContain('data-slot="insight-assessment"');
expect(moodBodySource).toContain(
@@ -288,7 +288,7 @@ export function MoodInsightsSections({
// v1.13.1 — match the canonical `gap-1.5` + `pb-1` heading-to-body
// rhythm the other assessment cards use, so the heading sits tight
// above its body instead of floating ~16-24 px above it.
className="animate-insight-in gap-2 py-3 md:py-4"
className="animate-insight-in gap-1.5 py-3 md:py-4"
>
<CardHeader>
<TileHeader