Skip to content

Credit IEM radar on-map + a11y/UX fixes#15

Merged
johncarmack1984 merged 1 commit into
mainfrom
audit/quickwins-a11y
Jun 27, 2026
Merged

Credit IEM radar on-map + a11y/UX fixes#15
johncarmack1984 merged 1 commit into
mainfrom
audit/quickwins-a11y

Conversation

@johncarmack1984

Copy link
Copy Markdown
Owner

First pass from the codebase audit — the safe, high-value quick wins.

  • Licensing fix: IEM NEXRAD (the live-radar fallback) is now credited in the on-map attribution control and the precipitation "live" status, not just the README. The repo's own rule treats an uncredited rendered source as a licensing bug. (audit: docs/attribution)
  • A11y — reduced motion: the wind layer drops its animated particles and renders the speed raster only under prefers-reduced-motion. (audit: ui-taste)
  • A11y — contrast: control panel bumped to bg-slate-900/90 for legibility over a light basemap. (audit: ui-taste)
  • UX: the control panel now caps its height and scrolls instead of clipping controls on short viewports. (audit: ui-taste)

tsc, biome, and the web build all pass locally.

Docs & attribution

  • On-map attribution updated (App.tsx + precip status); README already credited IEM.

…ed-motion

- Add IEM NEXRAD (the live-radar fallback) to the on-map attribution and the
  precipitation "live" status. It was credited in the README but not on the
  map, which the repo's own "credit in both surfaces" rule treats as a
  licensing bug.
- Cap the control panel height (max-h + overflow-y-auto + overscroll-contain)
  so it no longer clips controls on short viewports.
- Bump the panel to bg-slate-900/90 for legibility over a light basemap.
- Respect prefers-reduced-motion: the wind layer renders the speed raster only
  (no animated particles) when the user opts out of motion.
@johncarmack1984 johncarmack1984 merged commit 8acc80b into main Jun 27, 2026
7 checks passed
@johncarmack1984 johncarmack1984 deleted the audit/quickwins-a11y branch June 27, 2026 04:23
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