diff options
| author | Automerge <automerge@vaadin.com> | 2012-04-17 17:06:08 +0000 |
|---|---|---|
| committer | Automerge <automerge@vaadin.com> | 2012-04-17 17:06:08 +0000 |
| commit | c3e10ebbbc48d124a88d6e92e16faccdebe43223 (patch) | |
| tree | 8dbcf40a539c8a137ece117569467360a0eb5fc3 /src/com/vaadin/ui | |
| parent | 13563f900b809aabb997d1e7cb26efc6dc2bed6a (diff) | |
| download | vaadin-framework-c3e10ebbbc48d124a88d6e92e16faccdebe43223.tar.gz vaadin-framework-c3e10ebbbc48d124a88d6e92e16faccdebe43223.zip | |
[merge from 6.7] Reverted #7607 fixes and removed the test for now because they cause the #8662 regression.
svn changeset:23566/svn branch:6.8
Diffstat (limited to 'src/com/vaadin/ui')
| -rw-r--r-- | src/com/vaadin/ui/Table.java | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java index 88961e610a..e605ec4f6b 100644 --- a/src/com/vaadin/ui/Table.java +++ b/src/com/vaadin/ui/Table.java @@ -1261,11 +1261,14 @@ public class Table extends AbstractSelect implements Action.Container, maxIndex = 0; } - // Assume that we want to scroll to the very bottom (so that the bottom - // row is completely visible even if (table height) / (row height) is - // not an integer.) + /* + * FIXME #7607 Take somehow into account the case where we want to + * scroll to the bottom so that the last row is completely visible even + * if (table height) / (row height) is not an integer. Reverted the + * original fix because of #8662 regression. + */ if (newIndex > maxIndex) { - newIndex = maxIndex + 1; + newIndex = maxIndex; } // Refresh first item id |