jQuery Grid Plugin - jqGrid

Hello edallen!

I wanted to post following in the "bugs" part of forum, but after read your mail I deceide to make it here. It seems to me, that my solution is a little better.

I suggest insert in grid.base.js between lines 802 and 803:

$('.ui-pg-selbox',"#"+pgcnt).bind('change',function() {
    ts.p.rowNum = this.value;

following new line

    ts.p.page = Math.round(ts.p.rowNum*(ts.p.page-1)/this.value-0.5)+1;

(ts.p.rowNum*(ts.p.page-1) - is the number of rows currently skiped.

 ts.p.rowNum*(ts.p.page-1)/this.value - is the number of pages which sould be skipt, but as a float number. Math.round(x-0.5) is truncation instead of round. And +1 at the end becasue we count bages begins with 1)

I describe shortly what problem fixes the line.

Let as we have a grid with pager und rowList=[10,20,100]. At the beginning the first 10 rows of the data loaded and displayed. User press 2 times the button “next page” and stay on page 3 from 5 totally. Now user decides to scroll better in 20 steps or in 100 steps. He changes pager size (rowNum) from 10 to 20 or 100 and receive a message, that there are no data to display. The reason was that after changing the pager size (rowNum), the current page number (page) stays unchanged and so will be requested from server to load data from the wrong page. To fix the problem one should either reset page value to 1 or recomputed the page value.

Regards

Oleg

Hi Tony,

A small remark. In the current version on GitHub the lines 816 and 817 in grid.base.js should be swaped. Correct code should be:

            ...
            $('.ui-pg-selbox',"#"+pgcnt).bind('change',function() {
                ts.p.page = Math.round(ts.p.rowNum*(ts.p.page-1)/this.value-0.5)+1;
                ts.p.rowNum = this.value;
                ...

Best regards

Oleg