From: John Ahlroos Date: Thu, 25 Jul 2013 06:53:50 +0000 (+0300) Subject: Fixes light theme window sprites #12171 X-Git-Tag: 7.1.2~18 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cbab936d08b35414c4f1ce8bea55cebc53e7d4cf;p=vaadin-framework.git Fixes light theme window sprites #12171 The close buttons images for the Window black theme was mistakenly added to the white theme sprite and ended up in the middle of the light theme window. Moved the black theme button images to the black theme sprite. Change-Id: Ieaaaa1b404f1d623c61fe7cfecb4996d96904b27 --- diff --git a/WebContent/VAADIN/themes/reindeer/window/window.scss b/WebContent/VAADIN/themes/reindeer/window/window.scss index e8f0011397..57bd2d4c12 100644 --- a/WebContent/VAADIN/themes/reindeer/window/window.scss +++ b/WebContent/VAADIN/themes/reindeer/window/window.scss @@ -210,12 +210,12 @@ .#{$primaryStyleName}-closebox { top: 8px; - background-image: url(img/black/close.png); /** sprite-ref: verticals */ + background-image: url(img/black/close.png); /** sprite-ref: black-verticals */ &:hover { - background-image: url(img/black/close-hover.png); /** sprite-ref: verticals */ + background-image: url(img/black/close-hover.png); /** sprite-ref: black-verticals */ } &:active { - background-image: url(img/black/close-pressed.png); /** sprite-ref: verticals */ + background-image: url(img/black/close-pressed.png); /** sprite-ref: black-verticals */ } } .#{$primaryStyleName}-footer { @@ -232,23 +232,23 @@ .#{$primaryStyleName}-maximizebox { top: 8px; - background-image: url(img/black/maximize.png); /** sprite-ref: verticals */ + background-image: url(img/black/maximize.png); /** sprite-ref: black-verticals */ &:hover { - background-image: url(img/black/maximize-hover.png); /** sprite-ref: verticals */ + background-image: url(img/black/maximize-hover.png); /** sprite-ref: black-verticals */ } &:active { - background-image: url(img/black/maximize-active.png); /** sprite-ref: verticals */ + background-image: url(img/black/maximize-active.png); /** sprite-ref: black-verticals */ } } .#{$primaryStyleName}-restorebox { top: 8px; - background-image: url(img/black/restore.png); /** sprite-ref: verticals */ + background-image: url(img/black/restore.png); /** sprite-ref: black-verticals */ &:hover { - background-image: url(img/black/restore-hover.png); /** sprite-ref: verticals */ + background-image: url(img/black/restore-hover.png); /** sprite-ref: black-verticals */ } &:active { - background-image: url(img/black/restore-active.png); /** sprite-ref: verticals */ + background-image: url(img/black/restore-active.png); /** sprite-ref: black-verticals */ } } diff --git a/uitest/src/com/vaadin/tests/components/window/WindowThemes.html b/uitest/src/com/vaadin/tests/components/window/WindowThemes.html new file mode 100644 index 0000000000..a93c280984 --- /dev/null +++ b/uitest/src/com/vaadin/tests/components/window/WindowThemes.html @@ -0,0 +1,27 @@ + + + + + + +New Test + + + + + + + + + + + + + + + + + +
New Test
open/run/com.vaadin.tests.components.window.WindowLightDarkThemes
screenCapturewindows
+ + diff --git a/uitest/src/com/vaadin/tests/components/window/WindowThemes.java b/uitest/src/com/vaadin/tests/components/window/WindowThemes.java new file mode 100644 index 0000000000..2b39916db8 --- /dev/null +++ b/uitest/src/com/vaadin/tests/components/window/WindowThemes.java @@ -0,0 +1,62 @@ +/* + * Copyright 2000-2013 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ + +/** + * + */ +package com.vaadin.tests.components.window; + +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.Reindeer; + +public class WindowThemes extends AbstractTestUI { + + @Override + protected void setup(VaadinRequest request) { + Window def = new Window("default", new Label("Some content")); + def.setWidth("300px"); + def.setHeight("100%"); + addWindow(def); + + Window light = new Window("WINDOW_LIGHT", new Label("Some content")); + light.setStyleName(Reindeer.WINDOW_LIGHT); + light.setPositionX(300); + light.setWidth("300px"); + light.setHeight("100%"); + addWindow(light); + + Window black = new Window("WINDOW_BLACK", new Label("Some content")); + black.setStyleName(Reindeer.WINDOW_BLACK); + black.setPositionX(600); + black.setWidth("300px"); + black.setHeight("100%"); + addWindow(black); + } + + @Override + protected String getTestDescription() { + return "Shows the different css themes of Window"; + } + + @Override + protected Integer getTicketNumber() { + // Not tied to any specific ticket + return null; + } +}