Skip to content

fix(chart): legibility pass (supersedes #74 and #77)#81

Open
holoros wants to merge 1 commit into
mainfrom
feat/chart-legibility-v2
Open

fix(chart): legibility pass (supersedes #74 and #77)#81
holoros wants to merge 1 commit into
mainfrom
feat/chart-legibility-v2

Conversation

@holoros

@holoros holoros commented Jun 12, 2026

Copy link
Copy Markdown
Owner

What

Consolidated, more legible growth-chart pass, prompted by the chart still reading as heavy/small/cluttered on the live site.

  • Text size + contrast: axis ticks and the unit label go from 10 to 12 px and from low-contrast #8aa0b0 to #b6c6d0; end-gutter labels from 7.5 to 9.5 px with more vertical stacking room.
  • FIA observed label: now sits in a dark rounded pill so it stays readable over the data lines instead of colliding with the cream-colored series (the collision visible in the screenshots).
  • Floating stub: short anchor series (gcbm_moja_v6, 2022-2026) render as dots only — no phantom dashed segment in the corner.
  • Roomier layout: wider margins, slightly taller plot.
  • Folds in hover-to-highlight dimming and the CSV export.

Supersedes

This replaces #74 (stub markers) and #77 (hover-dim + CSV) with one cleaner chart change. Close #74 and #77 in favor of this; everything they did is here plus the legibility fixes.

Scope / risk

  • One file, src/GrowthChart.jsx. Presentation only; data, scrubber, isolate, PNG export untouched.
  • npm run build passes.

Opened for review — not auto-merged.

…/CSV

Addresses the growth chart reading as heavy/small/cluttered:
- axis tick + unit text larger (10->12) and higher contrast (#8aa0b0->#b6c6d0)
- FIA observed label now sits in a dark pill so it stays readable over
  the data lines instead of colliding with them
- end-gutter labels enlarged (7.5->9.5) with more stacking room
- short anchor stubs (gcbm_moja_v6, 2022-2026) render as dots only, so no
  stray floating dashed segment
- roomier margins + taller plot; folds in hover-to-highlight dimming and
  the CSV export. Build verified.

Supersedes #74 (stub fix) and #77 (hover-dim + CSV).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant