- Notifications
You must be signed in to change notification settings - Fork48
Comments
Addition: allow aria-hidden on picture element#352
Conversation
Revise the allowances for the `picture` element to allow for the `aria-hidden` attribute.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
This comment has been minimized.
This comment has been minimized.
marcoscaceres commentedOct 26, 2021
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. |
marcoscaceres commentedOct 26, 2021
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: |
Uh oh!
There was an error while loading.Please reload this page.
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 tow3c/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
Uh oh!
There was an error while loading.Please reload this page.
Revise the allowances for the
pictureelement to allow for thearia-hiddenattribute.Need at least two checkers to accept this change before we can merge.
Closes#346
Preview |Diff