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

Contrast minimum: Address the active (pressed) state #4127

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mfairchild365
Copy link

From #157: the active state (the typically split-second state when a button or control is being clicked or pressed) is not explicitly required to meet the contrast requirement.

This question comes up from time to time. The answer to this is provided in #157 but you have to dig to find it. Hopefully this change will make it easier to find the answer.

From w3c#157: the active state (the typically split-second state when a button or control is being clicked or pressed) is not explicitly required to meet the contrast requirement.
Copy link

netlify bot commented Nov 1, 2024

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 71fba36
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/6724d5b9392edd0008d026ff
😎 Deploy Preview https://deploy-preview-4127--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 site configuration.

@w3cbot
Copy link

w3cbot commented Nov 1, 2024

mfairchild365 marked as non substantive for IPR from ash-nazg.

@patrickhlauke
Copy link
Member

At the risk of reopening old wounds, I'm not sure the decision to somehow special-case/exempt "temporary" states isn't a slippery slope. And frankly I'm not sure why you'd want to have low-contrast active states to begin with. Might be worth bringing this up for discussion again, as I'm really not sure there's a defensible rationale (other than "back in 2014 the group said it's ok, so we'll honour that decision")

@stevefaulkner
Copy link

(other than "back in 2014 the group said it's ok, so we'll honour that decision")

not a good enough reason

@GreggVan
Copy link

GreggVan commented Nov 1, 2024 via email

@patrickhlauke
Copy link
Member

what is the need for the exemption? what is this trying to solve/allow? what sites would unnecessarily fail without this strangely specific exemption?

@mfairchild365
Copy link
Author

mfairchild365 commented Nov 1, 2024

I’m all for opening old wounds! 🤣

Honestly, I just created this pull request because in the linked issue it sounded like a settled question. Personally, I think it should fail normatively, but with a minimal impact / severity (of which WCAG has no concept of).

Im totally fine if we want to discuss this further rather than merge.

@patrickhlauke
Copy link
Member

If the idea is that contrast ratios don't apply to things in general that happen very quickly/for a short period of time - like a quick transition for instance - then it's worth defining this in broad terms, rather than just mentioning the ultra-specific case of :active state of something.

@mfairchild365
Copy link
Author

I don’t think this should apply to things in general that happen very quickly. Sometimes important information will flash on the screen (which may fail a different SC), but that content should still have sufficient contrast.

I think the argument here is that the element itself is not transitory. Instead it’s just a state that is transitory and attached to a permanent element that contains the same text / information that displays in the transitory state. Additionally, the state is triggered by the user and is displayed for only a brief moment before the element is removed. It’s only the moment in time when an element is being activated.

Again, normatively, I think this should fail. Should I update the PR to clarify that it fails?

@mraccess77
Copy link

It seems like updating the documentation on active state may require some sort of review by the wider group.

@patrickhlauke
Copy link
Member

In the first instance, we'll discuss it in the WCAG 2.x backlog meeting/subgroup, to then take it to the wider AGWG

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.

6 participants