design-reviews#361: LazyLoad (loading= attribute)

#361: LazyLoad (loading= attribute)

Visit on Github.

Opened Apr 3, 2019

Góðan dag TAG！

I'm requesting a TAG review of:

Name: LazyLoad
Specification URL: Pull request for spec is https://github.com/whatwg/html/pull/3752
Explainer, Requirements Doc, or Example code: Explainer: https://github.com/scott-little/lazyload
Tests: In progress
Primary contacts: scott-little, bengreenstein

Further details (optional):

Relevant time constraints or deadlines: We're aiming for Chrome M75, which would go to Beta around May 2nd - May 9th.
I have read and filled out the Self-Review Questionnare on Security and Privacy. The assessment is here.
I have reviewed the TAG's API Design Principles

We'd prefer the TAG provide feedback as (please select one):

open issues in our Github repo for each point of feedback
open a single issue in our Github repo for the entire review
leave review feedback as a comment in this issue and @-notify [github usernames]

Please preview the issue and check that the links work before submitting

For background, see our explanation of how to write a good explainer.

Discussions

Discussed Apr 24, 2019 (See Github)

Tess: suggest pushing this out - there is active discussion - good feedback - would rather review when it settles out

Peter: didn't chrome ship this?

Sangwhan: presto had this as a non-standard feature.

Peter: "Opera did it first" [meme]

Tess: it's been quite active - maybe pushing it a week would be the right amount of time.

Peter: let's push it a week

Discussed May 1, 2019 (See Github)

will postpone to another tim

Discussed May 8, 2019 (See Github)

Tess: Question on status of the pull request... the under-specifiedness of the auto behaviour. Seems to have petered out. We could poke and ask if they came to consensus. I will do that.

[agreed to put it to f2f

Comment by @hober May 8, 2019 (See Github)

Hi @scott-little,

What's the current status of the pull request? There was some active discussion around three weeks ago (starting with this comment from Jake) about how auto should work. Has that settled out? Is this stable enough to review?

Comment by @cynthia May 21, 2019 (See Github)

@hober and I discussed this during the Iceland F2F, and we think this is a really useful feature. We have jotted down a bunch of questions for ourselves to review when you have a slightly more stabilized proposal. Can you please nudge us again when you have an update?

Comment by @domfarolino Jul 26, 2019 (See Github)

Hey everyone, I've taken over the current spec PR, and the latest discussion has left its direction in a better state.

Here are some updates for this thread:

Per spec, images and iframes automatically have the "auto" LazyLoad state. This allows UAs to defer these, unless they are in-viewport, or not connected (in the DOM sense). This means that when these elements are created from JavaScript, they will not be deferred, which was the biggest thing folks were concerned about in situations like Jake's comment, for compatibility
The latest spec review happens to align the spec very closely to Chrome's implementation, which is just about ready to ship. We've been landing WPTs for this as well.
I will be updating the spec soon, to address the latest review

Let me know if I can clarify anything/provide more info

Comment by @hober Sep 12, 2019 (See Github)

Hi, sorry for the long delay. We're trying to make a dent in our backlog!

@cynthia and I are looking at this in our Tokyo F2F, and we're pretty happy with where this is, so we're going to close our review. If you decide to take a different direction, please file a new issue because we'd love to take a look at it.