-
Notifications
You must be signed in to change notification settings - Fork 320
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
Make dark grey darker #1827
Make dark grey darker #1827
Conversation
The palette on the colour page is magically* generated from the Sass, so will automatically change once updated to use the latest version of frontend 💥 * It's not really magic, it's just code, some gaffa tape and crossed fingers |
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.
LGTM
This'll need an entry in the changelog. |
326ec43
to
7e760e1
Compare
7e760e1
to
3775a1c
Compare
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.
LGTM 👍
3775a1c
to
579a7c5
Compare
The grey hint text continues to cause issues for users, with many struggling to read it or missing it because it's too pale. For an example see this comment on the text input component.
This PR changes
govuk-colour("dark-grey")
from#626a6e
to#505a5f
. This is the one of the palest greys I could find that meets AAA contrast for body copy:I appreciate we have darkened this grey before and that we probably shouldn't go any darker than this because it'll get so close to being black as to be pointless. This grey has a bit less contrast with the black than before, though I'd argue that this shouldn't affect the usability of form elements for those that can't detect the difference in colour.
This is the simplest change we can make now with the fewest repercussions across the system. If we continue to see problems after this change, we clearly can't make it any darker so will look into an alternative approach like bolding labels and making hint text black.
This change will need to be accompanied by changes to the colour page in the Design System.
Relates to alphagov/govuk-design-system#1222