-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Firefox 75: Add loading attribute to <img> element #5862
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
Updated data for `mozGetAsFile()` to indicate that it was deprecated in Firefox 26 and removed in 74. Source: * https://bugzilla.mozilla.org/show_bug.cgi?id=1588980 * https://www.fxsitecompat.dev/en-CA/docs/2013/htmlcanvaselement-mozgetasfile-has-been-deprecated/
This reverts commit 90728ee.
Firefox 75 adds the `<img>` attribute `loading`(and the corresponding property on `HTMLImageElement`'. Sources: * https://bugzilla.mozilla.org/show_bug.cgi?id=1542784 * https://bugzilla.mozilla.org/show_bug.cgi?id=1613611 * https://html.spec.whatwg.org/#attr-img-loading
api/HTMLImageElement.json
Outdated
| }, | ||
| "firefox_android": { | ||
| "version_added": false, | ||
| "notes": "See <a href='https://bugzil.la/1542784'>bug 1542784</a>" |
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.
This refers to a bug that is resolved fixed and I don't see information about Firefox Android there right away. Can you clarify why this bug reference is needed or is it the wrong bug?
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.
@a2sheppy This PR is still blocked on this question, if you get a chance to take a look soon. Thank you!
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.
@a2sheppy ping
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.
Change submitted. Sorry about that.
Removed the note from the `loading` attribute's entry for Firefox Android, as it doesn't really apply.
Elchi3
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.
Thanks!
loadingattribute to the HTMLImageElement pageloadingon Web performance resourcesSources: