Opened 10 years ago

Closed 7 years ago

#9128 closed enhancement (patchwelcome)

Add support for a paging grid

Reported by: Nathan Toone Owned by:
Priority: high Milestone: future
Component: DojoX Grid Version: 1.3.0
Keywords: Cc: jared_j, Josh Trutwin
Blocked By: Blocking:

Description

Per the article at http://dojotoolkit.org/2009/04/09/virtual-scrolling-grids-versus-paging-tables

This would be nice to have as an option to the grid itself - since we probably should support that functionality for people

Change History (7)

comment:1 Changed 10 years ago by Josh Trutwin

please as trutwijd to CC

comment:2 Changed 10 years ago by Nathan Toone

Cc: Josh Trutwin added

comment:3 Changed 10 years ago by dtynan

I did notice that there is a small issue with the paging grid when you have autoHeight set, rather than gridHeight, and you hit "Next" and so on to load results. The grid's height shrinks up to just a row or so & it displays the grid's "loading..." message. This looks pretty weird when it happens.

All you have to do to recreate this is change gridHeight="300" to autoHeight="4" (or whatever), in Jared's example page (http://jaredj.dojotoolkit.org/pageTable_example.html). I tried both Soria & Tundra, same results.

Just to be super clear, the grid's height will stay fixed as you hit Next, Next, etc. when gridHeight is set, but when using autoHeight, the grid's height shrinks & grows with each Next, Next, etc. (as it loads data).

Note there is an attempt to get rid of the "loading..." message near the top of the pageTable.js, but it doesn't seem to be preventing this (I've tried two recent nightly builds).

Thanks, Dylan Tynan

comment:4 Changed 10 years ago by dtynan

Also, setting gridHeight="-1" (as mentioned in the source) does not fully work either (plus how would I specify a grid of only 4 rows or 6 rows, etc.).

comment:5 Changed 9 years ago by Nathan Toone

Owner: Nathan Toone deleted

Unassigning my tickets.

comment:6 Changed 7 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of dgrid and gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:7 Changed 7 years ago by bill

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.