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

Remove F87 #2800

Closed
wants to merge 5 commits into from
Closed

Remove F87 #2800

wants to merge 5 commits into from

Conversation

awkawk
Copy link
Member

@awkawk awkawk commented Nov 16, 2022

Removing F87 Failure technique. Resolves #1554. Closes #3781

@JAWS-test
Copy link

F3 may also be removed soon, because CSS graphics can also get an alternative text. Possibly we should still wait for sufficient screen reader support

@awkawk awkawk requested review from alastc and patrickhlauke April 11, 2023 14:32
@awkawk
Copy link
Member Author

awkawk commented Apr 11, 2023

This came up again today in an audit. This Failure has a couple of fatal flaws:

  1. WCAG does not require that users can disable CSS. See https://www.w3.org/TR/UNDERSTANDING-WCAG/conformance.html#uc-conformance-claims-head where it indicates that CSS is one of the technologies that a web might identify as "relied upon".
  2. In the technique, it is indicating that sites cannot use before/after CSS content unless it is decorative. In the procedure 2 is "Verify that the content is decorative" (if this is false then the failure condition applies). This is counter to the conformance requirements. Procedure step 3 is redundant in that it only applies if step 2 is false, and the results indicate failure if 2 or 3 are false.

For a failure, it needs to represent a failure all of the time, and this clearly doesn't. We might say "browser X" doesn't support this, but WCAG doesn't require that every browser/AT provides support for any technology or technique: https://www.w3.org/TR/UNDERSTANDING-WCAG/conformance.html#uc-support-level-head

techniques/index.html Outdated Show resolved Hide resolved
techniques/index.html Outdated Show resolved Hide resolved
@patrickhlauke
Copy link
Member

due to the number of conflicts on this, I will redo this as a fresh PR, but we agreed in today's TF discussion that this is valid

@mbgower
Copy link
Contributor

mbgower commented May 3, 2024

I altered the description so it will close two different related issues on adoption. @patrickhlauke if you're remaking the PR, please ensure the same 2 are referenced.

patrickhlauke added a commit that referenced this pull request May 4, 2024
Rationale: this is not *always* a failure. User agent/screen reader support has improved considerably since the technique was first devised, with most (all?) combinations now happily reading/announcing CSS-generated content.

Supersedes #2800 (which was having too many merge conflicts due to its age)

Closes #1554, closes #3781
@patrickhlauke
Copy link
Member

Closing this in favor of #3818

@patrickhlauke patrickhlauke deleted the RemoveF87 branch May 4, 2024 10:51
mbgower pushed a commit that referenced this pull request Jun 12, 2024
Rationale: this is not *always* a failure. User agent/screen reader
support has improved considerably since the technique was first devised,
with most (all?) combinations now happily reading/announcing
CSS-generated content.

Supersedes #2800 (which was having too
many merge conflicts due to its age)

Closes #1554, closes #3781
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants