-
Notifications
You must be signed in to change notification settings - Fork 127
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
Conversation
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.
Other than fixing up that one id, looks good to me!
Thanks, @scottaohara!
@cookiecrook I'm guessing that you will enjoy reviewing this one. ;) |
Co-authored-by: James Craig <cookiecrook@users.noreply.github.com>
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.
Thank you for the addition!
fixes #758 |
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.
< 8000 p dir="auto">https://webkit.org/b/224615 |
Co-authored-by: James Craig <cookiecrook@users.noreply.github.com>
created a bug with chromium to get this implemented |
Using
none
as a reference point, created a first draft of theimage
entry into the spec.This initial commit only adds the note about why
image
is being added to the newimage
entry. Once we are happy with this note, we can add it to theimg
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