ReviewOS

also looking at this

stacks/commshq

fix(ui): align component palette and contain homepage hero

#16
Merged glennmichael123 wants to merge fix/commshq-component-theme into main
7 files +157 -13
10 days ago

Summary

  • Load low-specificity light/dark palette defaults through both UI and root STX configuration.
  • Map all component color and radius roles to the app palette while preserving marketing/auth overrides.
  • Remove the hero's overflowing negative margin and allow its grid column to shrink.
  • Add generated-CSS, stylesheet-wiring, and homepage-layout regression tests.

Verification

  • Local lint, typecheck, test, and build passed.
  • Full-page public UI and light/dark component contrast QA completed.
  • Browser measured zero horizontal overflow at 375, 768, and 1280px after the hero fix.
  • Authenticated dashboard workflows were not covered.

Release

Awaiting owner merge approval. Main triggers production deployment, so merge approval also needs to account for that deployment.

7 changed files on the files tab, with 0 review threads.

Sign in to comment on this pull request.