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

Fix for twitter style onscroll pagination #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

manuganji
Copy link

I had this issue. The page loads with a show_more link at the end. When show_more link is clicked, it's loading the next page of items via ajax but this doesn't happen automatically as it should be when we set paginateOnScroll:true. StackOverflow Question here

For on scroll pagination. It's not working with win.scroll. I switched it to doc.scroll and it's working fine. 

Also, I didn't see any reason 'doc' should not be a local variable instead of a global. so put a 'var' in front of it. 

These changes are working fine for my use case. i.e. twitter style on-scroll pagination.
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

Successfully merging this pull request may close these issues.

1 participant