-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: A11y - Use a darker contrast-compliant gray #396
Conversation
✅ Deploy Preview for cal-itp-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why our code formatting has been changing lately? Is it a missing plugin? Or a new version that formats differently?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 😅
Even for small PRs like this, we want to keep things as clean as possible!
closes #389
Use
tertiary-gray
instead ofseconday
:I'm going to keep the secondary variable for now, in case we use it for non-text things like borders, etc.