-
Notifications
You must be signed in to change notification settings - Fork 48
Addition: allow aria-hidden on picture element #352
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
Conversation
Revise the allowances for the `picture` element to allow for the `aria-hidden` attribute.
This comment has been minimized.
This comment has been minimized.
Note, you need to do the same for the other pull requests. I'll review those and make sure it's all good. Feel free to also ping me for review on those. However, we don't need to wait to merge things into the Editor's draft. The minute the spec gets published as REC, we can then start pushing all the ".addition" to /TR/. In the meantime, I've disabled the auto-publish.yml script. |
Actually, it's a little more nuanced: https://github.com/w3c/tr-design/blob/e40643f08e80f65a31d54ab2fb9e3ca34fa6cfcc/src/base.css#L35-L38
The things in the process are outlined in: |
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
directly reference the following PRs in the changelog, and prepend each entry with the appropriate 'label' related to w3c/html-aria#419 - w3c/html-aria#352 - w3c/html-aria#367 (and follow on [commit](w3c/html-aria@9cc2c0a)) - w3c/html-aria#360 - w3c/html-aria#353 - w3c/html-aria#372 - w3c/html-aria#402 - w3c/html-aria#404 - w3c/html-aria#403 - w3c/html-aria#396 - w3c/html-aria#391 - w3c/html-aria#369 - w3c/html-aria#381
Revise the allowances for the
picture
element to allow for thearia-hidden
attribute.Need at least two checkers to accept this change before we can merge.
Closes #346
Preview | Diff