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

SmooOothin it out #3

Open
rickydazla opened this issue Feb 28, 2015 · 7 comments
Open

SmooOothin it out #3

rickydazla opened this issue Feb 28, 2015 · 7 comments
Assignees

Comments

@rickydazla
Copy link
Member

@jayvin say I wanted to load the images before the scrollbar hits the bottom to make it smoother. I assume there is some kinds offset option I could mess with? Where it at??

@jayvin
Copy link
Contributor

jayvin commented Mar 2, 2015

@rickydazla , there is this one WaitForImagesToBeLoaded(true or false), but this option waits for all other next/previous page images to load before adding it to the page.
Here you mean, on initialization, the infinite scroll should wait that all images have been loaded then trigger the next/previous page loads, right?

@rickydazla
Copy link
Member Author

See "Setting Threshold" on this page: http://www.appelsiini.net/projects/lazyload

@jayvin
Copy link
Contributor

jayvin commented Mar 3, 2015

Currently the lazyload plugin is not used inside the infinite scroll. It has some other implementation like WaitForImagesToBeLoaded.
But for example in ZaraTerez whenever a page is loaded, in a callback function I do re-initialize the lazy load plugin for the images that have been just loaded.
So you want the lazyload plugin to be used inside the infinite scroll itself?

@rickydazla
Copy link
Member Author

That would be https://www.youtube.com/watch?v=qCbtXC0u-Jo 🍦

@jayvin
Copy link
Contributor

jayvin commented Mar 3, 2015

:) , So let's add the lazy load plugin into the infinite scroll and we will have settings for the lazyload that can be pushed to the plugin also. Something like this lazyLoadSettings:{Threshold:200,....}
What do you say? On which store we can test this out?

@mattmagi
Copy link

Did you guys ever implement this? 👍

@rickydazla
Copy link
Member Author

No. We got sidetracked... But we should!

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

3 participants