Skip to content

Amend "contrast ratio" term #4400

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

patrickhlauke
Copy link
Member

A different take compared to #1018

While reviewing that PR, it was noted that some of the existing notes in the term are inappropriate and incorrect.

This PR removes them altogether (which does mean this is a normative change), and moves one note to the understanding for contrast minimum and contrast enhanced.

The term now (with 1.4.11) applies to both text and non-text content, but it seems incorrect to refer to specific testing methodology in the normative term rather than in the more specific SCs. 1.4.3 and 1.4.6 understanding (https://www.w3.org/WAI/WCAG22/Understanding/contrast-minimum.html and https://www.w3.org/WAI/WCAG22/Understanding/contrast-enhanced) already have a note about antialiasing, so it's redundant to have here)
The note about assuming a particular background is contentious at best.

The note about "it is a failure" seems inappropriate in the normative term definition. A failure of what? This sort of note (if it were actually normatively intended) probably should go in the understanding document for a specific SC.
mentioning font smoothing here is irrelevant
Copy link

netlify bot commented May 17, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 0b7c816
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/682905b46c2c0f0008aa09d1
😎 Deploy Preview https://deploy-preview-4400--wcag2.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 project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant