mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-10-27 05:57:27 +00:00
rcourtman
526fa790ae
Stacked on #2909 (its commit is the parent here and lands with either PR). A full-resolution visual pass of the guest and node drawers, plus a Sol review of the same screenshots, found the charts and attention hard to read: - **Numeric axes.** History I/O axes read "Max / Avg / 0" with no magnitude, and "Avg" was the scale midpoint, not a measured average (demand ledger "Numeric axis labels on history I/O charts", #1720). Thermals read "Max / Mid / Min". Gridline labels now state the scale value (e.g. `5.0M/s`, `60°C`) and sit centred on their gridlines. - **Worth checking up front, one line each.** It was a full card per section below the charts. It is now a compact list directly under the alerts line, using the shared detail-row tone. - **Now vs history.** Trend card headlines say "now", and the range reads "24h low–high", so the live number is not read as the last point of the stored line. - **No text wall.** The two-sentence memory-cache caveat under the charts (repeated in Details) becomes "may include cache" on the Memory card; the full note stays once, in Details. - The missing-readings note is one line, and the Details summary uses readable text. Tests: the full frontend suite passes (15,467). New proofs: Worth checking renders above the trend cards with the shared tone, and the guardrail pins that the list reuses `detailValueToneClass`. Browser receipt: full-resolution screenshots of every drawer tab at 1440 and 390px, reviewed. Follow-ups not in this PR (from the same review): threshold lines on History charts, a crosshair shared across panels, taller History plots, a Manage tab layout pass, and status text beside the drawer's status dot.