Skip to content
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 colour contrast in publication headers #497

Merged
merged 1 commit into from
Oct 4, 2017

Conversation

vanitabarrett
Copy link
Contributor

@vanitabarrett vanitabarrett commented Oct 2, 2017

HTML publication headers currently don't meet WCAG AA colour contrast ratio for normal text. Changing this from $light-blue to $govuk-blue fixes this.

Before:
screen shot 2017-10-02 at 14 42 08

After:
screen shot 2017-10-02 at 14 42 15

Previous ratio was 3.72:1, new ratio is 6.68:1 (need to meet 4.5:1)

Live page: https://government-frontend-pr-497.herokuapp.com/government/publications/budget-2016-documents/budget-2016

Review app component guide:
https://government-frontend-pr-497.herokuapp.com/component-guide

Visual regression results:
https://government-frontend-pr-497.surge.sh/gallery.html

HTML publication headers currently don't meet WCAG AA colour contrast ratio for normal text. Changing this from $light-blue to $govuk-blue fixes this.
@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-497 October 2, 2017 14:50 Inactive
@vanitabarrett vanitabarrett requested a review from maxgds October 2, 2017 14:55
@NickColley
Copy link
Contributor

👍 has one of the designers from your team had a looksie?

@fofr
Copy link
Contributor

fofr commented Oct 4, 2017

Showed this to @alextea and @markhurrell, who seemed ok with it.

@vanitabarrett vanitabarrett merged commit bc2f14f into master Oct 4, 2017
@vanitabarrett vanitabarrett deleted the publication-banner-colour-contrast branch October 4, 2017 13:39
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.

4 participants