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

Change text color from #727272 to #595959 #25

Merged
merged 1 commit into from
May 2, 2016
Merged

Change text color from #727272 to #595959 #25

merged 1 commit into from
May 2, 2016

Conversation

AlexDaniel
Copy link
Contributor

As mentioned in Issue #1 this is the lightest color for gray text to pass WCAG

There are other places on the website where the contrast is not as good, but
these problems can be addressed later.

As mentioned in Issue #1 this is the lightest color for gray text to pass WCAG

There are other places on the website where the contrast is not as good, but
these problems can be addressed later.
@AlexDaniel
Copy link
Contributor Author

AlexDaniel commented Apr 19, 2016

↓ Before ↓

before

↓ After ↓

after

Visually not a big difference, but it is quite significant if you actually read the text.

@orderedlist orderedlist merged commit 6b0a5c2 into orderedlist:master May 2, 2016
@AlexDaniel AlexDaniel deleted the patch-1 branch May 13, 2016 00:39
@neuhaus
Copy link

neuhaus commented Oct 24, 2016

Thanks for merging this PR, however when I create a site using the generator with the minimal theme it still has the low contrast and the rgb #727272 color.

When will you roll out this change to the site generator?

tooomm added a commit to Cockatrice/cockatrice.github.io that referenced this pull request May 7, 2017
lightest color for gray text to pass WCAG (orderedlist/minimal#25)
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.

3 participants