Fix Editor overlay position when footer is displayed on top (#19491)

Change-Id: Ifffe66c8a872fe3625cf8c60115b172610b6e9b5

1 files changed, 21 insertions, 0 deletions

diff --git a/uitest/src/com/vaadin/tests/fieldgroup/BasicCrudGridEditorRowTest.java b/uitest/src/com/vaadin/tests/fieldgroup/BasicCrudGridEditorRowTest.java
index 769fa52323..df250357d8 100644
--- a/uitest/src/com/vaadin/tests/fieldgroup/BasicCrudGridEditorRowTest.java
+++ b/uitest/src/com/vaadin/tests/fieldgroup/BasicCrudGridEditorRowTest.java
@@ -15,6 +15,8 @@
  */
 package com.vaadin.tests.fieldgroup;
 
+import static org.junit.Assert.assertFalse;
+
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
@@ -89,4 +91,23 @@ public class BasicCrudGridEditorRowTest extends MultiBrowserTest {
         Assert.assertNotEquals("Checkbox value did not change", value,
                 cb.getValue());
     }
+
+    @Test
+    public void testNoTopStyleSetOnEditorOpenWithFooterOnTop() {
+        GridCellElement cell = grid.getCell(2, 3);
+        // Open editor row
+        new Actions(getDriver()).doubleClick(cell).perform();
+
+        // Close editor
+        new Actions(getDriver()).sendKeys(Keys.ESCAPE).perform();
+
+        cell = grid.getCell(14, 3);
+
+        // Open editor row
+        new Actions(getDriver()).doubleClick(cell).perform();
+
+        String attribute = grid.getEditor().getAttribute("style").toLowerCase();
+        assertFalse("Style should not contain top.", attribute.contains("top:"));
+    }
+
 }