diff options
3 files changed, 42 insertions, 1 deletions
diff --git a/WebContent/VAADIN/themes/valo/components/_window.scss b/WebContent/VAADIN/themes/valo/components/_window.scss index 23fa5338c2..52f57df183 100644 --- a/WebContent/VAADIN/themes/valo/components/_window.scss +++ b/WebContent/VAADIN/themes/valo/components/_window.scss @@ -110,6 +110,7 @@ $v-window-modality-curtain-background-color: #222 !default; $scroll-divider-width: max(1px, first-number($v-border)); .#{$primary-stylename}-outerheader { + cursor: move; position: absolute; z-index: 2; top: 0; @@ -133,7 +134,6 @@ $v-window-modality-curtain-background-color: #222 !default; } .#{$primary-stylename}-header { - cursor: move; line-height: $v-unit-size - 1px; padding-left: round($v-unit-size/3); margin-right: $v-unit-size * 2; diff --git a/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindow.java b/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindow.java new file mode 100644 index 0000000000..9783db4413 --- /dev/null +++ b/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindow.java @@ -0,0 +1,19 @@ +package com.vaadin.tests.themes.valo; + +import com.vaadin.annotations.Theme; +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUI; +import com.vaadin.ui.Label; +import com.vaadin.ui.Window; +import com.vaadin.ui.themes.ValoTheme; + +@Theme(ValoTheme.THEME_NAME) +public class NonDraggableWindow extends AbstractTestUI { + @Override + protected void setup(VaadinRequest request) { + Window window = new Window("Non-draggable window", new Label()); + window.setDraggable(false); + + addWindow(window); + } +} diff --git a/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindowTest.java b/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindowTest.java new file mode 100644 index 0000000000..f5a1bd903e --- /dev/null +++ b/uitest/src/com/vaadin/tests/themes/valo/NonDraggableWindowTest.java @@ -0,0 +1,22 @@ +package com.vaadin.tests.themes.valo; + +import static org.hamcrest.CoreMatchers.is; +import static org.junit.Assert.assertThat; + +import org.junit.Test; +import org.openqa.selenium.WebElement; + +import com.vaadin.testbench.By; +import com.vaadin.tests.tb3.MultiBrowserTest; + +public class NonDraggableWindowTest extends MultiBrowserTest { + + @Test + public void cursorIsDefault() { + openTestURL(); + + WebElement header = findElement(By.className("v-window-header")); + + assertThat(header.getCssValue("cursor"), is("default")); + } +}
\ No newline at end of file |