-
Notifications
You must be signed in to change notification settings - Fork 44
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
Safari doesn't load images within view on page load #500
Comments
This was with Safari 16.0. It does still happen with Safari Beta 16.3, but doesn't appear to happen with current Safari Tech Preview. So I assume it must be a bug in Safari they have fixed. Can't think of a workaround though, annoyingly. |
@dracos thanks a lot for reporting this strange behaviour. I can reproduce the problem even though that it's really strange and I would need to investigate further. |
It's strange because it works out of another context (JSBin instead of the website you've provided): |
@dracos by the way, what's the polyfills versions you're using ? |
On that page, it's 1.2.0. Replacing it with the one in your jsbin it does load them (though with a long pause). |
This polyfill has been working well for us for years, thanks (looks like I contributed to it at one point, in fact :) ). Now, in Safari 16 here, the images that are 'in view' at page load are not loaded. For example, see the first few images on https://www.theyworkforyou.com/mps/ or the ones around letter M if you go straight to https://www.theyworkforyou.com/mps/#M . This doesn't appear to be a bug in the polyfill, in that I can see the intersectionObservers existing, firing, removing the data and adding the correct src - but Safari does not then load the image. I'll post a ticket to Safari as well, but I'm posting here just in case I've missed something.
The text was updated successfully, but these errors were encountered: