Let column resize handle overflow its parent (#16838)

Change-Id: I795956a4512f1e6e688e1e3c0522cc952117d78f

3 files changed, 16 insertions, 9 deletions

diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnReorderTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnReorderTest.java
index 3ccca105de..6eb6d95643 100644
--- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnReorderTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnReorderTest.java
@@ -334,18 +334,18 @@ public class GridColumnReorderTest extends GridBasicFeaturesTest {
         GridCellElement draggedHeaderCell = getGridElement()
                 .getHeaderCell(0, 1);
         final int xOffset = 500;
-        new Actions(getDriver()).moveToElement(draggedHeaderCell, 5, 5)
+        new Actions(getDriver()).moveToElement(draggedHeaderCell, 10, 10)
                 .clickAndHold().moveByOffset(xOffset, 0).build().perform();
 
         WebElement floatingDragElement = findElement(By
                 .className("dragged-column-header"));
 
-        int expectedLeft = draggedHeaderCell.getLocation().getX() + xOffset + 5
-                - (floatingDragElement.getSize().getWidth() / 2);
+        int expectedLeft = draggedHeaderCell.getLocation().getX() + xOffset
+                + 10 - (floatingDragElement.getSize().getWidth() / 2);
         int realLeft = floatingDragElement.getLocation().getX();
 
         assertTrue("Dragged element location wrong, expected " + expectedLeft
-                + " was " + realLeft, Math.abs(expectedLeft - realLeft) < 5);
+                + " was " + realLeft, Math.abs(expectedLeft - realLeft) < 10);
     }
 
     @Test
diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnResizeTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnResizeTest.java
index 4345cc6f07..a2417bb02f 100644
--- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnResizeTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridColumnResizeTest.java
@@ -79,7 +79,14 @@ public class GridColumnResizeTest extends GridBasicFeaturesTest {
 
     @Test
     public void testResizeFirstColumn() {
-        dragResizeColumn(0, -10);
+        dragResizeColumn(0, -1, -10);
+        assertTrue("Log should contain a resize event",
+                logContainsText("ColumnResizeEvent: isUserOriginated? true"));
+    }
+
+    @Test
+    public void testDragHandleStraddlesColumns() {
+        dragResizeColumn(0, 4, -10);
         assertTrue("Log should contain a resize event",
                 logContainsText("ColumnResizeEvent: isUserOriginated? true"));
     }
@@ -87,19 +94,19 @@ public class GridColumnResizeTest extends GridBasicFeaturesTest {
     @Test
     public void testColumnPixelSizesSetOnResize() {
         selectMenuPath("Component", "Columns", "All columns auto width");
-        dragResizeColumn(0, -10);
+        dragResizeColumn(0, -1, -10);
         for (String msg : getLogs()) {
             assertTrue("Log should contain a resize event",
                     msg.contains("ColumnResizeEvent: isUserOriginated? true"));
         }
     }
 
-    private void dragResizeColumn(int columnIndex, int offset) {
+    private void dragResizeColumn(int columnIndex, int posX, int offset) {
         GridCellElement headerCell = getGridElement().getHeaderCell(0,
                 columnIndex);
         Dimension size = headerCell.getSize();
         new Actions(getDriver())
-                .moveToElement(headerCell, size.getWidth() - 1,
+                .moveToElement(headerCell, size.getWidth() + posX,
                         size.getHeight() / 2).clickAndHold()
                 .moveByOffset(offset, 0).release().perform();
     }
diff --git a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridSortingTest.java b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridSortingTest.java
index b471612798..548e8dbd2f 100644
--- a/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridSortingTest.java
+++ b/uitest/src/com/vaadin/tests/components/grid/basicfeatures/server/GridSortingTest.java
@@ -196,7 +196,7 @@ public class GridSortingTest extends GridBasicFeaturesTest {
     }
 
     private void clickHeader(GridCellElement headerCell) {
-        new Actions(getDriver()).moveToElement(headerCell, 5, 5).click()
+        new Actions(getDriver()).moveToElement(headerCell, 10, 10).click()
                 .perform();
     }