-
Notifications
You must be signed in to change notification settings - Fork 140
Add role=image as synonym for role=img #1370
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
carmacleod
left a comment
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 <[email protected]>
mcking65
left a comment
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.
Co-authored-by: James Craig <[email protected]>
|
created a bug with chromium to get this implemented |
Using
noneas a reference point, created a first draft of theimageentry into the spec.This initial commit only adds the note about why
imageis being added to the newimageentry. Once we are happy with this note, we can add it to theimgsection 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