petoendre / google-web-toolkit-incubator

Automatically exported from code.google.com/p/google-web-toolkit-incubator
0 stars 0 forks source link

PagingScrollTable doesn't work w/o paging #158

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of gwt and gwt-incubator are you using?
gwt 1.5.2 and latest released incubator jar (1.5_sep_04)

What OS and browser are you using?
irrelevant

Do you see this error in hosted mode, web mode, or both?
both

The javadoc on setPageSize says:
By default, the page size is zero, which indicates that all rows should be 
shown on the page.

This doesn't seem to be the case now. If pageSize == 0 then in setData 
method firstVisibleRow == 0 (pageSize * currentPage) and lastVisibleRow == 
-1 ((currentPage + 1) * pageSize - 1). [assuming this is the first page and 
table model returns -1 as number of rows is unknown]
With these params and firstRow being 0 for the fist page currRow in 
VisibleRowIterator is equal to 0 and always greater than lastVisibleRow 
being -1. Which makes the table to skip rendering all the data.

Possible solution would be to make lastVisibleRow be zero ... but I still 
can't figure out what params should I supply to make this happen.

Original issue reported on code.google.com by ata...@gmail.com on 5 Oct 2008 at 1:08

GoogleCodeExporter commented 9 years ago

Original comment by ecc%google.com@gtempaccount.com on 1 Dec 2008 at 5:56

GoogleCodeExporter commented 9 years ago
committed as r1588

The PagingScrollTable now requests all rows on the 0th page instead of 
requesting 0 rows.

Original comment by jlaba...@google.com on 25 Mar 2009 at 3:31