-
Notifications
You must be signed in to change notification settings - Fork 287
Time based media EO references #4339
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
base: AudioDescriptionPreexisting
Are you sure you want to change the base?
Conversation
Thanks @mbgower ! I only had time for a quick skim. A minor request: "W3C Web Accessibility Initiative" -> "W3C Web Accessibility Initiative (WAI)" Will this get propagated to Understanding docs for 2.1 & 2.2, too? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
understanding/20/audio-description-or-media-alternative-prerecorded.html
Outdated
Show resolved
Hide resolved
Adjusted as per request
Done
We do not touch the 2.0 space, but it will appear in 2.2 and 2.1 versions. |
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
fixes #4341 by addressing minor typos
resolves #4341 outstanding typos as well. |
There seem to remain a number of defunct links to techniques like the SMIL techniques - not sure if they produce a 404 error due to the way I rendered the file or whether this is to be left to another PR. |
Which page are you talking about? On the current live page for captions (prerecorded), those seem fine. |
I was talking about "Understanding Audio Description or Media Alternative (Prerecorded)" - I just went to "Files changed" in this PR and then "View file" and put the URL of that through Githack with this result - but in that view all the Techniques URLs don't work - so forget it... |
note that thanks to the magic of @kfranqueiro you don't need to manually go via githack or similar ... just use the netlify deploy preview #4339 (comment) … https://deploy-preview-4339--wcag2.netlify.app/ in your specific case https://deploy-preview-4339--wcag2.netlify.app/understanding/audio-description-or-media-alternative-prerecorded |
fulfils request from @shawna-slh :
I added an items to the Resources section of the A and AA Understanding docs, and in AAA where there was material that seemed to be specific to the SC. Six in total.
Also
resolves #4341
I added in some trivial corrections to the 1.2 Time-based media understanding doc to close a recent issue.