Skip to content
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

Add role=image as synonym for role=img #1370

Merged
merged 5 commits into from May 6, 2021
Merged

Add role=image as synonym for role=img #1370

merged 5 commits into from May 6, 2021

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Dec 15, 2020

Using none as a reference point, created a first draft of the image entry into the spec.

This initial commit only adds the note about why image is being added to the new image entry. Once we are happy with this note, we can add it to the img section as well... if we think that's necessary (they do show up right next to each other in the spec, after all).


Preview (#image) (#img) | Diff

Copy link
Contributor

@carmacleod carmacleod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than fixing up that one id, looks good to me!
Thanks, @scottaohara!

index.html Outdated Show resolved Hide resolved
@carmacleod
Copy link
Contributor

@cookiecrook I'm guessing that you will enjoy reviewing this one. ;)

index.html Outdated Show resolved Hide resolved
Co-authored-by: James Craig <cookiecrook@users.noreply.github.com>
Copy link
Contributor

@mcking65 mcking65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@scottaohara

Thank you for the addition!

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Base automatically changed from master to main January 20, 2021 22:59
@carmacleod carmacleod linked an issue Mar 4, 2021 that may be closed by this pull request
@jnurthen
Copy link
Member

jnurthen commented Mar 4, 2021

fixes #758

@jnurthen jnurthen added the Agenda label Mar 4, 2021
@jnurthen
Copy link
Member

Decision in 25-Mar-2021 meeting - no fallback role

revise the wording per matt’s feedback.  Removed the paragraph on fallback per the decision the WG came to in the march 25th call.
@cookiecrook
Copy link
Contributor

https://webkit.org/b/224615

@jnurthen jnurthen merged commit d75a248 into main May 6, 2021
jnurthen pushed a commit that referenced this pull request May 6, 2021
Co-authored-by: James Craig <cookiecrook@users.noreply.github.com>
@pkra pkra deleted the issue758 branch January 10, 2022 16:53
@pkra pkra added this to the ARIA 1.3 milestone Jan 10, 2022
@pkra pkra mentioned this pull request Jan 10, 2022
@jcsteh
Copy link

jcsteh commented Apr 21, 2023

@scottaohara
Copy link
Member Author

created a bug with chromium to get this implemented
cc @aleventhal @benbeaudry https://bugs.chromium.org/p/chromium/issues/detail?id=1463426

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add role="image" as synonym for role="img"
8 participants