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

Specify <iframe loading=lazy> #5496

Closed
zcorpan opened this issue Apr 29, 2020 · 3 comments · Fixed by #5579
Closed

Specify <iframe loading=lazy> #5496

zcorpan opened this issue Apr 29, 2020 · 3 comments · Fixed by #5579
Assignees
Labels
addition/proposal New features or enhancements

Comments

@zcorpan
Copy link
Member

zcorpan commented Apr 29, 2020

#3752 originally included the loading attribute for both img and iframe. This was shipped in Chrome. I think after Chromium's intent to ship (for both img and iframe) was approved, we narrowed the scope of the spec PR to only specify loading for img.

It appears there is interest from Gecko and WebKit for the iframe case as well, but it not being specified is a problem:

https://bugzilla.mozilla.org/show_bug.cgi?id=1622090
https://bugs.webkit.org/show_bug.cgi?id=196698

I think now seems like a good time to specify <iframe loading=lazy>.

@zcorpan zcorpan added the addition/proposal New features or enhancements label Apr 29, 2020
@domfarolino domfarolino self-assigned this Apr 30, 2020
@domfarolino
Copy link
Member

Thanks for filing this! I'm happy to take this. I think #5236 should probably a blocker for this, since it is mostly about creating some Intersection Observer infrastructure that we reuse for images and iframes (and other things) in a more reusable and interoperable way.

@addyosmani
Copy link

I think now seems like a good time to specify <iframe loading=lazy>.

Thanks, @zcorpan! It's time... :) From the Chrome side:

  • We are excited & committed to contribute to spec-work for <iframe loading=lazy> (+1 to Lazy load intersection-checking should not be done in-parallel #5236 being a blocker we should resolve)
  • We intend to move the current implementation for native iframe lazy-loading behind a flag until comfortable the spec is in a good place. Currently looking at doing this for Chrome 84/85.
  • We strongly recommend developers avoid using <iframe loading=lazy> until this work is further along. Folks who have adopted native lazy-loading for iframes will soon find it a noop.

@domfarolino
Copy link
Member

We intend to move the current implementation for native iframe lazy-loading behind a flag until comfortable the spec is in a good place. Currently looking at doing this for Chrome 84/85.

Thanks @addyosmani. Just to clarify, it seems that given our latest blink-dev update, we won't be moving our implementation of iframe lazy-loading behind a flag, while developers are still discouraged from using it until it is fully standardized.

@domfarolino domfarolino mentioned this issue May 26, 2020
12 tasks
domenic pushed a commit that referenced this issue Jul 8, 2020
mfreed7 pushed a commit to mfreed7/html that referenced this issue Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addition/proposal New features or enhancements
Development

Successfully merging a pull request may close this issue.

3 participants