jQuery Grid Plugin - jqGridForum

Topic RSS

Performance improvements for scrolling in grid

25/03/2011
15:25

Klaus

Member

Members

Forum Posts: 68

Member Since:
18/11/2009

Offline

Hello,

I've detected performance problem when scrolling in big grids, because the browser checks every row if it's visible.

Could this be improved the following way, or are their any circumstances where this solution does not work?

populateVisible: function() {

...

var table = $("table:first", grid.bDiv);

- var rows = $("> tbody > tr:gt(0):visible:first", table);
- var rh = rows.outerHeight() || grid.prevRowHeight;

+ var rows = $("> tbody > tr:eq(1)", table);
+ var rh = rows.length ? (rows.outerHeight() || grid.prevRowHeight) : grid.prevRowHeight;

    // avoid slow selector (performance). Row 0 is the invisible "jqgfirstrow"
    // so row 1 (eq(1)) is the first visible row, we can therefor optimize this selector
    // and avoid numRowsInTable :visible checks which are very slow

...

Or is there a better solution?

Kind regards,

Klaus.

29/03/2011
17:22

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Thanks Klaus,

Make a better change (imho).

See here:

https://github.com/tonytomov/jqGrid/commit/217fa14cbb1a804095cd78eeb595072c380b8191

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:
74 Guest(s)

Currently Browsing this Page:
1 Guest(s)