Nelsoncardoso / jquery-datatables-column-filter

Automatically exported from code.google.com/p/jquery-datatables-column-filter
0 stars 0 forks source link

sFilterRow selection in a multi-table page. #82

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Use datetimepicker jquery plugin (anywhere on page) 
2. It creates a div with a fleshed out table for jui_datepicker 
(ui-datepicker-calendar)
3. When selecting sPlaceHolder to be "head:before". The "value" inside filter 
text boxes is [ Su, Mo... ].

Using v0.9.0, dataTables v1.9.2 on Ubuntu 12.04 x64 (chromium)

I have fixed the problem by making the tr selection in line 255 more specific, 
instead of just """tr = $("thead tr:first")""" (it doesn't specify which table 
to select the thead from)

Changed line 255 and 256 to 

   var tr = $("#"+oTable[0].id+"thead tr:first").detach();
   tr.prependTo("#"+oTable[0].id+"thead");

This way it selects the right thead. Chose to detach "tr:first" instead of 
"tr:last" because dataTables was picking "tr:last" to be the column-sorting row 
and giving problems in my specific case. (changing sort order whenever I click 
on filter text box to enter text).

Great plugin BTW, keep up the good work!

Original issue reported on code.google.com by reet...@v3r.us on 1 Aug 2012 at 8:45

GoogleCodeExporter commented 9 years ago
Hi,

Thanks for this fix, but could you plase use the latest version of plugin from 
the live examples? The version you have used is far too old. Current version is 
1.4.8 and probably this issue is resolved there.

Thanks,
Jovan

Original comment by joc...@gmail.com on 4 Sep 2012 at 7:57

GoogleCodeExporter commented 9 years ago
Hey Jovan,

It took me a while to realize that the file that is linked on the home page's 
Featured Downloads section (archive with examples and stuff) doesn't contain 
the latest version inside it. Tried the latest version and yes it works fine 
without having to modify the code. 

Cheers,

Original comment by reet...@v3r.us on 5 Sep 2012 at 3:11