Hide using display:none to be screenshot compatible (#10080)

Change-Id: I67b3e21640760ee6581b656450e0a5517788580d

diff --git a/WebContent/VAADIN/themes/tests-components/styles.css b/WebContent/VAADIN/themes/tests-components/styles.css
index 5b41f1824faf3da5b18a568d21b49010843d54eb..0680e2b47239bf962313f56dfcf3ac809cb6b12e 100644 (file)
--- a/WebContent/VAADIN/themes/tests-components/styles.css
+++ b/WebContent/VAADIN/themes/tests-components/styles.css
@@ -53,4 +53,11 @@
        box-shadow: inset 0 1px 2px rgba(0,0,0,.2);
        border-radius: .5em;
        background: rgba(0,0,0,.02);
+}
+
+.displaynone {
+       height: 0;
+       width: 0;
+       display: none;
+}
 }
\ No newline at end of file

diff --git a/uitest/src/com/vaadin/tests/components/AbstractComponentTest.java b/uitest/src/com/vaadin/tests/components/AbstractComponentTest.java
index c086e03ae017789aaa7917585d052ac353d8450b..8dae56b0795d9bb3abbc085671f94a5b19515804 100644 (file)
--- a/uitest/src/com/vaadin/tests/components/AbstractComponentTest.java
+++ b/uitest/src/com/vaadin/tests/components/AbstractComponentTest.java
@@ -165,12 +165,10 @@ public abstract class AbstractComponentTest<T extends AbstractComponent>
         // This is only to be screenshot-compatible with Vaadin 6, where
         // invisible components cause spacing
         if (visible) {
-            log.setHeight(null);
-            log.setWidth(null);
+            log.removeStyleName("displaynone");
             log.setCaption((String) log.getData());
         } else {
-            log.setHeight("0px");
-            log.setWidth("0px");
+            log.addStyleName("displaynone");
             log.setCaption(null);
         }
     }