Skip to content

C41: fix change of contrast ratio in Procedure #4422

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 2 commits into
base: main
Choose a base branch
from

Conversation

momdo
Copy link
Contributor

@momdo momdo commented May 31, 2025

https://www.w3.org/WAI/WCAG22/Techniques/css/C41#tests

The required change of contrast for Focus Appearance (Minimum) is 3:1, this technique goes slightly beyond the minumum requirement.

The success criterion "Focus Appearance (Minimum)" was renamed "Focus Appearance," and "Focus Appearance (Enhanced)" was deleted.
Therefore, the change of contrast ratio shown in the procedure should be 3:1 instead of 4.5:1.

Copy link

netlify bot commented May 31, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 8b067ec
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/683d96d550b3f10008b1f2da
😎 Deploy Preview https://deploy-preview-4422--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.

@patrickhlauke
Copy link
Member

while we're here ... the hex values in the code sample don't match the image that's then shown in the example.

The code sample uses

background-color: #236AB8; /* medium-dark blue */
outline: 3px #fff5be solid; /* light yellow */

but the example in the image actually uses

background-color: #003366; /* dark blue */
outline: 3px #ffff00 solid; /*yellow */

Suggest changing the example code as well (yes, the example then has a contrast ratio of 11.7:1 ... but this still gives a good example of a strong focus indicator)

Co-authored-by: Patrick H. Lauke <[email protected]>
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.

2 participants