-
Notifications
You must be signed in to change notification settings - Fork 13
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
Do not use any GIF placeholder for faster loading (omit src
and srcset
)
#56
Comments
But I am not sure why one occurrence hard codes the placeholder instead of using the given apply_filter. |
fwiw, Lazysizes also uses a transparent gif which is |
Actually, the LazyLoad author strongly suggest not to use any placeholder at all. I tested this myself and perceived performance is so much better without a placeholder because Progressive JPEG (and even baseline JPEG) shows so much faster. See: |
Sorry, correct link is: |
src
and srcset
)
I'm going to integrate this in the next minor version to see how it feels and if we get any feedbacks. Thank you for your contribution and pointing to this reference in the lazyload repo. |
@Tabrisrp Current lazyload js version had a severe SEO issue, see verlok/vanilla-lazyload#215 (comment) Just to inform you so you can update the plugin which should be just replacing the JS version. |
Tested version 10.11 and noted a bug where all images on the page are loaded directly instead of being lazy, so I reported it to verlok/vanilla-lazyload#225 and waiting for a fix before creating a new version. |
'src="data:image/gif;base64,R0lGODdhAQABAPAAAP///wAAACwAAAAAAQABAEACAkQBADs="
sets a white 1x1 GIF as placeholder. This looks really ugly if the background is anything other than white. A transparent GIF would be much better: https://css-tricks.com/snippets/html/base64-encode-of-1x1px-transparent-gif/That is,
data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7
Hope you can change this to transparent and also update the docs at https://docs.wp-rocket.me/article/130-manually-apply-lazyload-to-an-image
The text was updated successfully, but these errors were encountered: