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

Lazy loading for iframes #840

Closed
zcorpan opened this issue Jun 30, 2023 · 4 comments
Closed

Lazy loading for iframes #840

zcorpan opened this issue Jun 30, 2023 · 4 comments

Comments

@zcorpan
Copy link
Member

zcorpan commented Jun 30, 2023

Request for Mozilla Position on an Emerging Web Specification

Other information

Also see #151

@zcorpan
Copy link
Member Author

zcorpan commented Jul 6, 2023

We support lazy-loaded images, and the feature makes sense for iframe for similar reasons. Initial page load is faster and more lightweight if out-of-view iframes are lazy-loaded. It's possible for sites to implement lazy-loading with JS but a declarative solution is easier to use and allows browsers to control when to load things.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1622090#c10 for an example of real-world impact.

Suggested position: positive.

@emilio
Copy link
Collaborator

emilio commented Jul 6, 2023

I agree fwiw.

@zcorpan zcorpan added ready to add Appears ready to add to the table of positions. position: positive labels Jul 6, 2023
@martinthomson
Copy link
Member

@zcorpan did you intend to add this to the dashboard?

@CanadaHonk
Copy link

This will soon be landed and most likely shipped, could we close this out?

@martinthomson martinthomson removed the ready to add Appears ready to add to the table of positions. label Oct 18, 2023
zcorpan added a commit that referenced this issue Oct 19, 2023
tantek pushed a commit that referenced this issue Oct 24, 2023
* Change lazy-loading-attr to also cover iframes

See #840

* Apply suggestions from code review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants