From 1a0a3bd5cf248d3b368e52608d38767f2a501115 Mon Sep 17 00:00:00 2001 From: Anna Koskinen Date: Mon, 4 Feb 2013 13:22:59 +0200 Subject: Merge of (#10890) to Vaadin 7. Fix the positioning of the drop marker when reordering Table columns by DnD. Change-Id: I853e9690d931a9bf94c61ac6e59accd1cedac15a --- client/src/com/vaadin/client/ui/VScrollTable.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'client/src') 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) { -- cgit v1.2.3