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

Pager with ajax: Change of pagesize #198

Closed
a--f opened this issue Dec 18, 2012 · 3 comments
Closed

Pager with ajax: Change of pagesize #198

a--f opened this issue Dec 18, 2012 · 3 comments
Labels

Comments

@a--f
Copy link

a--f commented Dec 18, 2012

Hey everyone,

first of all I'd like to thank you for tablesorter. It really is a great plugin making life much easier.
In reply to issue #182, I'm afraid to report that this fix introduced a new bug when using pager with ajax: Every time you change the page size and the pager plugin sees no need to change the current page, no ajax call is made. For example you start on the first page with 2 entries per page and change to 5 entries per page the table does not referesh to show those 5 entries but stays with the 2. But when you first change to page 2 and then to 5 entries per page, the table refreshes as expected. I guess you have to consider whether the page size changed or not in Line 333 in jquery.tablesorter.pager.js

Cheers
Andy

@Mottie
Copy link
Owner

Mottie commented Dec 18, 2012

Thanks for reporting this! I've actually already modified the code to match dhamma's pull request which should fix this problem.

@a--f
Copy link
Author

a--f commented Dec 18, 2012

Great! Looking forward to seeing this in action.

Cheers
Andy

@Mottie
Copy link
Owner

Mottie commented Dec 18, 2012

Ok, this should now be fixed in v2.6. Thanks for reporting this issue!

@Mottie Mottie closed this as completed Dec 18, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants