Change cursor to default for nondraggable windows in Valo. (#15377)

Change-Id: Ie019abbda1664e8dd4f53a76f2c6f21e3bc5443b

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;

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);
+    }
+}

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"));
+    }
+}