diff options
author | Anna Koskinen <anna@vaadin.com> | 2013-02-04 13:22:59 +0200 |
---|---|---|
committer | Anna Koskinen <anna@vaadin.com> | 2013-02-05 16:46:44 +0200 |
commit | 1a0a3bd5cf248d3b368e52608d38767f2a501115 (patch) | |
tree | c19fb1040a4f919839966522c41de0d0c06716e6 /client | |
parent | 1431139cb24af12edc6685a2841460a0d4dd03b9 (diff) | |
download | vaadin-framework-1a0a3bd5cf248d3b368e52608d38767f2a501115.tar.gz vaadin-framework-1a0a3bd5cf248d3b368e52608d38767f2a501115.zip |
Merge of (#10890) to Vaadin 7.
Fix the positioning of the drop marker when reordering Table columns by
DnD.
Change-Id: I853e9690d931a9bf94c61ac6e59accd1cedac15a
Diffstat (limited to 'client')
-rw-r--r-- | client/src/com/vaadin/client/ui/VScrollTable.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/client/src/com/vaadin/client/ui/VScrollTable.java b/client/src/com/vaadin/client/ui/VScrollTable.java index b9244a1e91..8553398718 100644 --- a/client/src/com/vaadin/client/ui/VScrollTable.java +++ b/client/src/com/vaadin/client/ui/VScrollTable.java @@ -2629,7 +2629,11 @@ public class VScrollTable extends FlowPanel implements HasWidgets, for (int i = start; i <= visibleCellCount; i++) { if (i > 0) { final String colKey = getColKeyByIndex(i - 1); - slotX += getColWidth(colKey); + // getColWidth only returns the internal width + // without padding, not the offset width of the + // whole td (#10890) + slotX += getColWidth(colKey) + + scrollBody.getCellExtraWidth(); } final int dist = Math.abs(x - slotX); if (closestDistance == -1 || dist < closestDistance) { |