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

A11y: Contrast of code formatting too low #2446

Open
hugovk opened this issue May 8, 2024 · 0 comments · May be fixed by #2491
Open

A11y: Contrast of code formatting too low #2446

hugovk opened this issue May 8, 2024 · 0 comments · May be fixed by #2491
Labels
frontend Relates to the frontend of the app

Comments

@hugovk
Copy link
Member

hugovk commented May 8, 2024

Describe the bug

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://www.python.org/downloads/release/python-3130a6/
  2. View the list of removed "dead batteries" with code formatting

Actual behavior

The contrast of the green text on a grey background is too low, making it hard to read and affecting accessibility.

It's something like 2.88:1.

https://webaim.org/resources/contrastchecker/?fcolor=11A611&bcolor=F1F2F3

Expected behavior

The ratio should be at least 4.5:1.

WCAG 2.0 level AA requires a contrast ratio of at least 4.5:1 for normal text and 3:1 for large text. ... WCAG Level AAA requires a contrast ratio of at least 7:1 for normal text and 4.5:1 for large text.

AA is a good target to aim for.

Screenshots

image

Desktop (please complete the following information):

  • OS: macOS
  • Browser Chrome
  • Version 124

Additional context

image

The contrast of the blue links in the table with different grey backgrounds is also a little low, would be good to bump these to 4.5:1+ as well, but the green above is more important.

@JacobCoffee JacobCoffee added the frontend Relates to the frontend of the app label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Relates to the frontend of the app
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants