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 load embeds with facades #1299

Closed
asafm7 opened this issue Jun 11, 2024 · 4 comments
Closed

Lazy load embeds with facades #1299

asafm7 opened this issue Jun 11, 2024 · 4 comments

Comments

@asafm7
Copy link

asafm7 commented Jun 11, 2024

Was this considered?

https://developer.chrome.com/docs/lighthouse/performance/third-party-facades/

@westonruter
Copy link
Member

The problem is that there is a very bad UX problem with façades, specifically for video. On mobile they require a second tap to play. See paulirish/lite-youtube-embed#6

@asafm7
Copy link
Author

asafm7 commented Jun 11, 2024

Interesting, @westonruter. Thanks.

I have my own implementation that I thought was working but now I see isn't auto-playing on iOS.

BTW, it might be worth mentioning in the plugin's description how what it does is different (and better) than just adding loading="lazy" to the iframe.

@westonruter
Copy link
Member

BTW, it might be worth mentioning in the plugin's description how what it does is different (and better) than just adding loading="lazy" to the iframe.

@asafm7 Well, that is what it does if the embed merely consists of an iframe. If there are scripts too then it will do additional work to delay loading them.

@swissspidy
Copy link
Member

We're already tracking this in #113, so closing as a duplicate.

@swissspidy swissspidy closed this as not planned Won't fix, can't repro, duplicate, stale Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants