Skip to content

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

Merged
merged 8 commits into from
Oct 26, 2021
Merged

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Sep 20, 2021

Revise the allowances for the picture element to allow for the aria-hidden attribute.

Need at least two checkers to accept this change before we can merge.

Closes #346


Preview | Diff

Revise the allowances for the `picture` element to allow for the `aria-hidden` attribute.
@scottaohara scottaohara added the needs implementation commitment Cannot merge into spec until implementations in conformance checkers has been confirmed. label Sep 20, 2021
@scottaohara scottaohara added this to the ARIA in HTML: Next milestone Sep 20, 2021
@scottaohara

This comment has been minimized.

@scottaohara scottaohara removed this from the ARIA in HTML: Next milestone Sep 24, 2021
@scottaohara scottaohara added the Meets 2 implementations req Indicates that a spec change has met the necessary 2 implementations needed to merge. label Oct 8, 2021
@scottaohara scottaohara removed the needs implementation commitment Cannot merge into spec until implementations in conformance checkers has been confirmed. label Oct 23, 2021
@marcoscaceres
Copy link
Member

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
Copy link
Member

Actually, it's a little more nuanced:

https://github.com/w3c/tr-design/blob/e40643f08e80f65a31d54ab2fb9e3ca34fa6cfcc/src/base.css#L35-L38

 *   - .correction for "candidate corrections"       (div, aside, details, section)
 *   - .addition   for "candidate additions"         (div, aside, details, section)
 *   - .correction.proposed for "proposed corrections" (div, aside, details, section)
 *   - .addition.proposed   for "proposed additions"   (div, aside, details, section)

The things in the process are outlined in:
https://www.w3.org/Consortium/Process/Drafts/#revising-rec

scottaohara and others added 2 commits October 26, 2021 11:34
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
@scottaohara scottaohara merged commit 223bf38 into gh-pages Oct 26, 2021
@scottaohara scottaohara deleted the issue-346 branch October 26, 2021 15:38
@scottaohara scottaohara changed the title allow aria-hidden on picture element May 19, 2022
scottaohara added a commit that referenced this pull request May 19, 2022
directly reference the following PRs in the changelog, and prepend each entry with the appropriate 'label'

related to #419

- #352
- #367 (and follow on [commit](887c772))
- #360
- #353
- #372
- #402
- #404
- #403
- #396
- #391
- #369
- #381
pkra pushed a commit to w3c/aria that referenced this pull request May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Meets 2 implementations req Indicates that a spec change has met the necessary 2 implementations needed to merge.
3 participants