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

Increase contrast for .status_prompt style #84

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

dontcallmedom
Copy link
Member

needed to reach WCAG-enough contrast over white background

needed to reach WCAG-enough contrast over white background
@netlify
Copy link

netlify bot commented Aug 21, 2023

Deploy Preview for w3c-typography ready!

Name Link
🔨 Latest commit d9b0d72
🔍 Latest deploy log https://app.netlify.com/sites/w3c-typography/deploys/64e333c88c837b0008366a7c
😎 Deploy Preview https://deploy-preview-84--w3c-typography.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dontcallmedom dontcallmedom requested review from xfq and r12a August 21, 2023 09:52
@xfq
Copy link
Member

xfq commented Sep 7, 2023

Would you please provide a before and after screenshot for our reference, @dontcallmedom?

@dontcallmedom
Copy link
Member Author

before
after

(changing WCAG contrast ratio from 2.32 to 4.71 according to WebAIM contrast checkr)

@r12a
Copy link
Contributor

r12a commented Sep 7, 2023

@xfq i had already done some tests for this suggested change, and i think it will probably be ok, even though it doesn't de-emphasise the affected text as much as i'd have liked. I'm assuming that Dom has picked the lightest grey colour that passes WAI AA - i didn't check that.

@xfq
Copy link
Member

xfq commented Sep 8, 2023

Thanks, @dontcallmedom and @r12a!

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