jQuery Grid Plugin - jqGridForum

Topic RSS

Related Topics sp_TopicIcon

showHideCol: cells[i] with large index can produce exception in IE

Tags: groupping showCol hideCol

10/01/2012
19:12

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

I tested some code used groupping with groupColumnShow: [false] setting. If one removes the groupping dynamically the method showHideCol will be called internally. In the lines 2970-2972 of grid.base.js will be enumerated all rows of grid:

$($t.rows).each(function(j){
$(this.cells[i]).css("display", show);
});

The rows having grouping information have not so many columns and the native implementation of the this.cells[i] produce exception in IE9. One can fix the problem by skipping of rows having "jqgroup" class:

$($t.rows).each(function () {
if (!$(this).hasClass("jqgroup")) {
$(this.cells[i]).css("display", show);
}
});

Best regards
Oleg

11/01/2012
10:27

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Thanks Oleg,

Fixed in GitHub

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
61 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

OlegK: 1255

markw65: 179

kobruleht: 144

phicarre: 132

YamilBracho: 124

Renso: 118

Member Stats:

Guest Posters: 447

Members: 11373

Moderators: 2

Admins: 1

Forum Stats:

Groups: 1

Forums: 8

Topics: 10592

Posts: 31289

Newest Members:

, razia, Prankie, psky, praveen neelam, greg.valainis@pa-tech.com

Moderators: tony: 7721, Rumen[Trirand]: 81

Administrators: admin: 66