"Video element auditory content has captions" [f51b46]: Adding 1.2.1 as a secondary requirement. #2089
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WilcoFiers
approved these changes
Jul 20, 2023
ChrisLoiselle
approved these changes
Jul 20, 2023
kengdoj
reviewed
Jul 20, 2023
_rules/video-captions-f51b46.md
Outdated
| @@ -20,6 +20,8 @@ accessibility_requirements: | |||
| failed: not satisfied | |||
| passed: further testing needed | |||
| inapplicable: further testing needed | |||
| wcag20:1.2.1: # Audio-only and Video-only (Prerecorded) (A) | |||
| secondary: true | |||
There was a problem hiding this comment.
Suggested change
| secondary: true | |
| secondary: Because this rule does not consider that the video may be a media alternative for text, it is stricter than the requirement. Failing this rule often results in this success criterion being not satisfied. |
Or something like that.
Using https://github.com/act-rules/act-rules.github.io/pull/2060/files as a guide.
There was a problem hiding this comment.
Changed this text to be the same was what we ended up deciding in the 2060 PR.
Jym77
changed the title
Jym77
approved these changes
Aug 10, 2023
Jym77
approved these changes
Aug 31, 2023
WilcoFiers
approved these changes
Aug 31, 2023
|
@tbostic32 I do think this needs a call for review. Please update that part of it. |
kengdoj
approved these changes
Aug 31, 2023
carlosapaduarte
approved these changes
Sep 15, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding 1.2.1 as a secondary requirement.
TF review sheet link (note many of the comments here are out of date)
Closes issue(s):
Need for Call for Review:
This will not require a Call for Review
How to Review And Approve