Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#12540 closed defect (fixed)

Enhanced Grid Paginator: Page-Stepper initialization problem

Reported by: manklu Owned by: evan
Priority: high Milestone: 1.7
Component: DojoX Grid Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

Enhanced Grid Paginator: Page-Stepper initialization problem

The code in Pagination._createPageStepNodes doesn't work correctly if the result of _getStartPage() is greater than zero.

For a quick fix, change the line 

for(var i = startPage; i < this.maxPageStep + 1; i++){

to 

for(var i = startPage; i < this.maxPageStep + startPage + 1; i++){

Copy the file 'test_enhanced_grid_pagination1.html' to 'dojox/grid/tests/enhanced' to see the before and after behaviour.

Attachments (3)

test_enhanced_grid_pagination1.html​ (3.5 KB) - added by manklu 8 years ago.
Testpage
actualResult.bmp​ (24.5 KB) - added by manklu 8 years ago.
Actual Result
expectedResult.bmp​ (23.1 KB) - added by manklu 8 years ago.
Expected result

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by manklu

Attachment: test_enhanced_grid_pagination1.html​ added

Testpage

Changed 8 years ago by manklu

Attachment: actualResult.bmp​ added

Actual Result

Changed 8 years ago by manklu

Attachment: expectedResult.bmp​ added

Expected result

comment:1 Changed 8 years ago by evan

Milestone: tbd → 1.7
Priority: normal → high

comment:2 Changed 8 years ago by evan

Resolution: → fixed
Status: new → closed

(In [25561]) Fixes #12540 !strict go to correct default page after Grid get rendered, also change 'pageSize' to '_pageSize' since it's internal use only.

comment:3 Changed 8 years ago by evan

(In [25562]) Refs #12540 !strict so that we can use plugins:{pagination:{defaultPage:3}} to set a default page

comment:4 Changed 8 years ago by evan

(In [25563]) Fixes #13265 !strict so that we can set default page size and default page to be displayed. Refs #12540. Also updated at ​http://docs.dojocampus.org/dojox/grid/EnhancedGrid/plugins/Pagination

comment:5 Changed 8 years ago by evan

(In [25673]) Refs #12540 !strict validate the pageSizes[] parameter, an invalid integer or integer < 0 indicates 'all'

Note: See TracTickets for help on using tickets.