lotjuh on ShowHideCol function bug

Wed, 21 Oct 2009 05:13:36 +0300

Thx for the hint. Will do that from now on.

Might be smart in the long term to make sure you're working with integers when you're expecting them though (-:

markw65 on ShowHideCol function bug

Tue, 20 Oct 2009 12:56:29 +0300

Unfortunately you'll have to fix a lot of other cases too. colModel[i].width is assumed to be an integer all over the place.

The best fix (unless tony goes to the trouble of fixing every use of colModel width) is to supply integer values for the widths...

Mark

lotjuh on ShowHideCol function bug

Tue, 20 Oct 2009 11:30:23 +0300

Hi Tony,

While playing with the showCol and hideCol functions, I stumbled upon a bug which resides both in the 3.5.x and 3.6 beta versions.

The bug:

I have 6 columns from which the first 2 are shown and the last 4 are hidden. I use a slider to dynamically rotate through the hidden and shown columns. When I slide my slider 1 step to the right, column 1 will be hidden and column 3 will be shown. Then when I slide my slider another step to the right, column 2 will be hidden and column 4 will be shown and my grid width will be resized to some really big size all of a suddon.

I've been following this with firebug, and I see the following happen in the ShowHideCol function:

$t.p.tblwidth is currently 500

this.width is currently "16"

$t.p.tblwidth += this.width is called

$t.p.tblwidth is now 50016

My solution:

replace the code in ShowHideCol : function(colname,show) :

if(show == "none") $t.p.tblwidth -= this.width; else $t.p.tblwidth += this.width;

with

if(show == "none") $t.p.tblwidth -= parseInt(this.width); else $t.p.tblwidth += parseInt(this.width);

Kind regards Lotte-Sara Laan

jQuery Grid Plugin - jqGrid - Topic: ShowHideCol function bug

lotjuh on ShowHideCol function bug

markw65 on ShowHideCol function bug

lotjuh on ShowHideCol function bug