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

Non-text contrast update #2028

Merged
merged 3 commits into from
Sep 15, 2021
Merged

Non-text contrast update #2028

merged 3 commits into from
Sep 15, 2021

Conversation

alastc
Copy link
Contributor

@alastc alastc commented Sep 7, 2021

Changes from the doc agreed today.

Preview

Closes #1775

@alastc alastc merged commit 79b3471 into main Sep 15, 2021
@alastc alastc deleted the Issue1775-non-text-contrast-focus branch September 15, 2021 21:26
@alastc alastc restored the Issue1775-non-text-contrast-focus branch September 17, 2021 12:06
<figure id="figure-focus-background">
<img src="img/ntc-focus-background.png" alt="Three blue buttons, the center button is a lighter blue than the others." width="400" />
<figcaption>
The change of background within the component is not in scope of non-text contrast. However, this would not pass <a href="../20/use-of-color.html">Use of color</a>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

didn't notice this at the time, but revisiting the google doc: while the change of just the background would fail use of color, the text of the button changes from white to black - the most dramatic change in contrast, which would not count as a change of color alone...so this would not fail Use of color?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

made this into a fresh issue #2061

@fstrr fstrr deleted the Issue1775-non-text-contrast-focus branch January 13, 2023 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants