Merge pull request 'fix(detail): give Results Over Time chart more height on desktop' (#78 ) from fix/detail-trend-chart-taller into main

Reviewed-on: #78
fix(detail): give Results Over Time chart more height on desktop
2026-07-21 21:08:34 +00:00 · 2026-07-21 22:05:10 +01:00
2 changed files with 8 additions and 2 deletions
@@ -855,7 +855,10 @@
 /* Charts Section */
 .chartContainer {
    width: 100%;
-    height: 280px;
+    /* Taller on desktop so the trend lines have vertical room to separate
+       and read clearly. Mobile overrides this to height:auto below (the
+       max-width:768px query), so this only affects desktop. */
+    height: 380px;
    position: relative;
 }

@@ -490,7 +490,10 @@
 /* ── Charts & Map ────────────────────────────────────── */
 .chartContainer {
    width: 100%;
-    height: 280px;
+    /* Taller on desktop so the trend lines have vertical room to separate
+       and read clearly. Mobile overrides this to height:auto below (the
+       max-width:768px query), so this only affects desktop. */
+    height: 380px;
    position: relative;
 }
Author	SHA1	Message	Date
tudor	d02a323cdc	Merge pull request 'fix(detail): give Results Over Time chart more height on desktop' (#78 ) from fix/detail-trend-chart-taller into main Stage (build -> staging -> E2E gate) / Build Backend (FastAPI) (push) Successful in 14s Details Stage (build -> staging -> E2E gate) / Build Frontend (Next.js) (push) Successful in 58s Details Stage (build -> staging -> E2E gate) / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 14s Details Stage (build -> staging -> E2E gate) / Deploy to Staging (push) Successful in 1s Details Stage (build -> staging -> E2E gate) / E2E Journeys against Staging (push) Successful in 1m13s Details Reviewed-on: #78	2026-07-21 21:08:34 +00:00
TudorandClaude Opus 4.8	32f8a02862	fix(detail): give Results Over Time chart more height on desktop PR Checks / Frontend Typecheck + Tests (pull_request) Successful in 1m6s Details PR Checks / Backend Smoke (pull_request) Successful in 7s Details PR Checks / Build Backend (no push) (pull_request) Successful in 11s Details PR Checks / Build Frontend (no push) (pull_request) Successful in 47s Details PR Checks / Build Pipeline (no push) (pull_request) Successful in 11s Details PR Checks / AI Code Review (Claude) (pull_request) Successful in 13s Details The trend chart was capped at 280px tall on desktop, squishing the lines together and making them hard to read. Raise the base .chartContainer height to 380px in both the primary and secondary detail views. Mobile is unaffected: the base value is already overridden to height:auto by the existing max-width:768px query, where PerformanceChart's own .chartWrapper carries the canvas height. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0146VHeLAWjDVE2B5uU67jCB	2026-07-21 22:05:10 +01:00