-
Notifications
You must be signed in to change notification settings - Fork 53
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
Does Turbolinks work with jQuery Infinite Pages? #5
Comments
It does not work with Turbolinks, the next page link is getting cached so if you link off the page and then link back to it the scrolling will continue to hit the last page number it had cached. |
I'm using jquery-infinite-pages with kaminari and simple_calendar to try and apply infinite scroll to the calendar so that on scroll it renders the next month. I've got this working for 4 months then when I get to June the link_to_next_page is gone and therefore the pagination doesn't scroll/paginate. Is there a preset configuration for maximum # of pages it will paginate? If so, how can I increase this amount. Thanks! |
I'm using this with turbolinks just fine. |
What is your test case? The container gets cached so if you turbolink off the page and then turbolink back onto the pages which uses jquery-infinite-pages then you will continually scroll the same end page. If you don't ever page off the current page or do a hard refresh then you are ok. |
@JaredCowan ... I'm having the same issue with a development piece I'm working on, how were you able to overcome this issue with turbolinks? |
I too have been unable to get this working in development. I've followed the readme and I seem to not get this working asynchronously. When I load the page up it doesn't even attempt to configure the table. |
This PR works for me: #15 |
The two larger PRs for fixing this didn't work for me, but I got it working with a one-line change that just unbinds the |
Hi |
@TrishaPatel I don't use this any more but it worked well with my one-line fix in #24 for the whole time I did. Does that fix work for you? |
This should be investigated...
The text was updated successfully, but these errors were encountered: