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

Replace misleading image in 1.4.11 understanding with clearer example, tweak all other button example images with bigger text #2574

Merged
merged 2 commits into from Aug 23, 2023

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Aug 8, 2022

the current image to illustrate how a simple change of background colour is not in scope of 1.4.11 non-text contrast actually muddies things, because the focused button also has a different text colour and a white halo/shadow around the text.

the current confusing image

replacing the image with a cleaner example image that only changes the background colour - and, to keep it more consistent with the other examples, it uses the light blue for the non-focused buttons and the dark blue for the focused one.

the amended example image

Closes #2494

EDIT: in addition, this also makes all "Example" text in the button images bigger (21px) and bold, to avoid any doubt that they'd pass 1.4.3 as "large text"

the current image to illustrate how a simple change of background colour is not in scope of 1.4.11 non-text contrast actually muddies things, because the focused button also has a different text colour and a white halo/shadow around the text.
this example *only* changes the background colour - and, to keep it more consistent with the other examples, it uses the light blue for the non-focused buttons and the dark blue for the focused one.

Closes #2494
@patrickhlauke patrickhlauke changed the title Replace misleading image with clearer example Replace misleading image in 1.4.11 understanding with clearer example Aug 8, 2022
To avoid doubt about whether or not the "white text on light blue" example buttons counts as large text and therefore pass with just a 3:1 contrast ratio, made all text in the buttons bigger (21px) and bold.
@patrickhlauke patrickhlauke changed the title Replace misleading image in 1.4.11 understanding with clearer example Replace misleading image in 1.4.11 understanding with clearer example, tweak all other button example images with bigger text Aug 9, 2022
@patrickhlauke
Copy link
Member Author

patrickhlauke commented Aug 9, 2022

expanded this PR to also tweak all "Example" blue buttons, making their text bigger/bold - see #2494 (comment)

@patrickhlauke
Copy link
Member Author

@alastc any chance this could be discussed and hopefully merged at some point soon?

@patrickhlauke
Copy link
Member Author

as this example has just caused some confusion in a WAI-IG discussion...can this please be looked at soon? @alastc @mbgower

@detlevhfischer
Copy link
Contributor

detlevhfischer commented Jul 14, 2023

@patrickhlauke

EDIT: in addition, this also makes all "Example" text in the button images bigger (21px) and bold, to avoid any doubt that they'd pass 1.4.3 as "large text"

Since size of text in images is always difficult to judge across platforms, I wonder why we can't have an example that meets 4,5:1? Apart from the muddying aspects noted, this was the case in the old example.

@patrickhlauke
Copy link
Member Author

patrickhlauke commented Jul 14, 2023

@detlevhfischer if somebody has the time/inclination to redo all the example images again ... sure. my change here was a quick patch over the existing images. redoing the backgrounds to a darker shade for 4.5:1 will likely require a more thorough/proper redoing of them, which frankly I have no time or energy for... i'd say let's merge this to fix the blatantly wrong/misleading example first and foremost though, rather than holding this up longer to get it "perfect"

@mbgower
Copy link
Contributor

mbgower commented Jul 18, 2023

@detlevhfischer I suggest we could add a note to the Understanding stating somethnig like 'all image examples of buttons are intended to represent 14pt bold text, and so only require a 3:1 contrast ratio against the button background in order to pass Contrast (Minimum)'

@alastc alastc merged commit c6b912f into main Aug 23, 2023
@alastc alastc deleted the patrickhlauke-issue2494 branch August 23, 2023 13:22
@patrickhlauke
Copy link
Member Author

@alastc @iadawn I've just noticed that the version on https://w3c.github.io/wcag/understanding/non-text-contrast.html is still showing the old (and, in the case of figure 15, incorrect) version (even though https://www.w3.org/WAI/WCAG22/Understanding/non-text-contrast.html is correct). any idea why the github.io version lags behind the actual repo content?

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.

Understanding Success Criterion 1.4.11: Non-text Contrast - Figure 15
5 participants