-
Notifications
You must be signed in to change notification settings - Fork 754
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 plugin when the main table contains tables in a cell #196
Comments
Thanks, I'll look into this. |
I forgot to include a fix in v2.6, but I'll have it done in the next update! Thanks again. |
Have you got any tips on how one might fix this now? I have a similar issue; a table is inside the footer. I tried adding the cssInfoBlock class to that table, but this didn't help. Sorting doesn't work for all columns when this bug is present. |
I'll have the plugin updated within the hour :) |
Hi, thanks for your quick work. You are great. However I think my bug is perhaps separate to this. It's not really related to the pager plugin which I incorrectly read here. The issu is this; I just downloaded the new zip for this project, and changed the demo table in index.html to contain a section with this:
So you see there is a nested table there. When this happens, the sorting gets messed up (for the first 2 columns). Please try this out and let me know if you can reproduce this (Safari). Thanks. |
@wesleyh That is a separate issue. I was about to go to bed, but my OCD kicked in... if you change line #321 from this: trs = $(t).find('thead:eq(0) tr, tfoot tr'), to this trs = $(t).find('thead:eq(0), tfoot').children('tr'), it'll work properly. I'll include this change in the next update. Thanks! |
If i have a 'main' table where one column has 'sub' tables in the cell contents
With the pager plugin when choosing the number of rows to display it seem to include the sub tables in the count so end up with too few rows
The text was updated successfully, but these errors were encountered: