From 8fcde69c2a970b9f175e0c599ab2bc2dc12b0b49 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Fri, 7 Dec 2012 16:35:07 +0200 Subject: [PATCH] Hide using display:none to be screenshot compatible (#10080) Change-Id: I67b3e21640760ee6581b656450e0a5517788580d --- WebContent/VAADIN/themes/tests-components/styles.css | 7 +++++++ .../com/vaadin/tests/components/AbstractComponentTest.java | 6 ++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/WebContent/VAADIN/themes/tests-components/styles.css b/WebContent/VAADIN/themes/tests-components/styles.css index 5b41f1824f..0680e2b472 100644 --- 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 c086e03ae0..8dae56b079 100644 --- 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 // 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); } } -- 2.39.5