-
Notifications
You must be signed in to change notification settings - Fork 139
Images NOT showing if you use browser Back button to go back to the previous page with yall lazy loaded images #38
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
Comments
Do these images reappear if you scroll or otherwise interact with the page after hitting the back button? In any event, I'll need a minimally reproducible test page (or pages in this case) to see the effect in action. |
https://docs.google.com/document/d/1gIxTtMXDW4ggoIJ-Ts1y33zlLX-ZeiCVg0EjbQszyFQ/edit?usp=sharing Once you open that URL page in Chrome, we load a bunch of images using yall with the following settings:
Then, just click on Sign-up on the top right corner. I dont feel like it has anything to do with the idlyLoad issue. Thanks for your help. |
Try dropping the attached version in to see if this helps: I need to test this more in other browsers, but when I use local sources in Chrome to test this on a local version of your site, it appears to work. It adds the |
Hi malchata, Thanks for the updated file. I tried it locally, and it seems to fix this issue. I expect it will be fine when deployed. Will let you know if we see any issues. Thanks again. |
Great. I'll get a new release prepared and close this. Thanks for getting
back to me!
…On Mon, Jan 14, 2019 at 3:53 AM Badhumvee ***@***.***> wrote:
Hi malchata, Thanks for the updated file. I tried it locally, and it seems
to fix this issue. I expect it will be fine when deployed. Will let you
know if we see any issues. Thanks again.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#38 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABjzpoW0EjXWF8fYTUh_-kakZ9KEyHVvks5vDFOsgaJpZM4Zph67>
.
|
Officially fixed and released as version 2.2.1. |
- Fixed issue where order of src/srcset could cause Safari (and possibly other browsers) to download redundant resources. - Fixed #38. - Updated packages. - Tweaked README. - Build is slightly smaller now.
Images NOT showing if you use browser Back button to go back to the previous page with yall lazy loaded images. That previous page loaded fine initially, but then you click on a link and navigate to second page ... then you hit browser back button (chrome and edge) the images that you saw on the first page are not visible anymore and instead, in developer tools you see the 1px-1px placeholder.jpg image in place of the real image.
Maybe we are doing something wrong. Could you please advise? Thanks a lot.
TS
The text was updated successfully, but these errors were encountered: