-
Notifications
You must be signed in to change notification settings - Fork 284
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
colVis + stateSave leads to broken filters #265
Comments
I also have this behavior with colVis and stateSave. |
please provide a link to a jsbin/jsfiddle/ etc so I can reproduce this scenario... |
@vedmack Thank you for your attention, here is a demo: http://live.datatables.net/kitawawo/4/edit?html,js,output To reproduce:
|
@jonnsn @1nickt , both of you are using datatables with DOM source? because if you do so then it didn't work only because I wasn't listening to the colvis events in DOM source mode, I just added it on my dropbox file and the jsbin works fine, let me know if it works for both of you and I'll commit it. |
Hi, I won't get a chance to review tonight, but no, I am using JavaScript -nick
|
@vedmack : I use DOM source and it seems to fix the issue for me. Thank you very much for your help! |
Hi again Daniel, Fixed! ( Using Javascript Data Source ) See http://live.datatables.net/kitawawo/8/edit I also tested with my real site. You are the man. Thanks very much. By the way, I tried many ways to get column filter boxes at the top of my
On Thu, Dec 10, 2015 at 5:12 PM, Daniel Reznick notifications@github.com
|
Fixed in 0.8.9.beta.27 grab it and see how it works for you. |
Hi,
I have a Datatable with colVis Button and stateSave option set to true.
Once I hide one or more columns and reload the page, the columns are still hidden therefore. If I want to show the columns now, there are no filters for them and I can't seem to add them on 'column-visibility.dt'.
I tried this for example:
but obviously appendFilters is not accessible here and I could not find another possibility to init the "new" columns.
I'd be happy for any suggestions :-)
The text was updated successfully, but these errors were encountered: