classtranscribe / FrontEnd

The React + Redux Frontend for ClassTranscribe
https://classtranscribe.illinois.edu
Other
25 stars 27 forks source link

Button color accessibility #757

Open tdy opened 6 months ago

tdy commented 6 months ago

Some buttons have insufficient text contrast. For some reason we have two different button implementations:

  1. Enough contrast: Styled automatically via src/layouts/CTButtons/styles.js
  2. Not enough contrast: Styled manually via src/components/stylesheets/general.vars.css

We could fix option 2 in isolation, but ideally we would just unify all buttons to use option 1.

angrave commented 6 months ago

@tdy I've recently updated general.vars and made other changes; it is possible that this issue is now fixed..