ColVis 1.0.8 - 1.1.0 "Show All" - Button Problems

jliebermann

hello again,
i've tried to use the Show All Button (not documented in the API but there sincd 2y / 1.0.7. i have 2 issues:

one is the usage of indexOf which will not work in IE <=8 (yes i know it's rather old but i have some xp clients). it's in the _fnDomShowAllButton function, around line 571 (637 in 1.1.0 dev). fixed it for me by using jQuery.inArray.

second is that the ShowAll function doesn't trigger a state change (like hiding/showing one column does). fixed it by adding the code to trigger the event...
[code]
"_fnDomShowAllButton": function ()
{
var
that = this,
nButton = document.createElement('button'),
nSpan = document.createElement('span');
nButton.className = !this.s.dt.bJUI ? "ColVis_Button TableTools_Button" :
"ColVis_Button TableTools_Button ui-button ui-state-default";
nButton.appendChild( nSpan );
$(nSpan).html( ''+this.s.sShowAll+'' );

$(nButton).click( function (e) {
for ( var i=0, iLen=that.s.abOriginal.length ; i

breenj

Nice fix jliebermann, had the same issue with IE8, your solution saved me alot of time :)