From b54b95dbc7d14c9f278418b7a349029b4525af6f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Thu, 4 Jun 2026 16:35:54 +0200 Subject: [PATCH] fix(insights): bound the mood time-of-day chart height The "mood by time of day" card spans the full overview width, unlike its distribution/weekday siblings which sit in a two-column grid. Sizing the chart wrapper with aspect-[3/2] derived the height off the full card width, so on a wide viewport the chart ballooned to roughly 800px tall and overflowed its card. Swap the aspect ratio for a fixed bounded height so the chart fits the card at the same size as the width-constrained sibling charts regardless of viewport. Chart internals and tokens are unchanged. --- src/components/insights/mood/mood-time-of-day-chart.tsx | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/components/insights/mood/mood-time-of-day-chart.tsx b/src/components/insights/mood/mood-time-of-day-chart.tsx index 7478ead1f..7c2d43d19 100644 --- a/src/components/insights/mood/mood-time-of-day-chart.tsx +++ b/src/components/insights/mood/mood-time-of-day-chart.tsx @@ -77,7 +77,12 @@ export function MoodTimeOfDayChart({ return (
-
+ {/* A fixed height, not `aspect-[3/2]`: this card spans the full overview + width (its weekday/distribution siblings sit in a 2-col grid and are + width-constrained), so an aspect ratio derived the height off the full + card width and ballooned the chart to ~800px on a wide viewport. A + bounded height keeps it the same size as the sibling charts. */} +