From c9586c93b93987d9b5942c38ef0b536c091092a5 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sauli=20T=C3=A4hk=C3=A4p=C3=A4=C3=A4?= Date: Fri, 6 Feb 2015 23:59:35 +0200 Subject: [PATCH] Hide button caption overflow in reindeer and valo. (#11864) Change-Id: Iaeb9cd571dd4b7a37cdc89f9d291365aef96c9e1 --- .../reindeer/button/button-standard.scss | 2 ++ .../themes/valo/components/_button.scss | 2 ++ .../tests/themes/ButtonTextOverflow.java | 27 +++++++++++++++++++ .../tests/themes/ButtonTextOverflowTest.java | 25 +++++++++++++++++ 4 files changed, 56 insertions(+) create mode 100644 uitest/src/com/vaadin/tests/themes/ButtonTextOverflow.java create mode 100644 uitest/src/com/vaadin/tests/themes/ButtonTextOverflowTest.java diff --git a/WebContent/VAADIN/themes/reindeer/button/button-standard.scss b/WebContent/VAADIN/themes/reindeer/button/button-standard.scss index 632340f228..4809611442 100644 --- a/WebContent/VAADIN/themes/reindeer/button/button-standard.scss +++ b/WebContent/VAADIN/themes/reindeer/button/button-standard.scss @@ -42,6 +42,8 @@ background-repeat: no-repeat; background-position: right top; background-image: url(img/right.png); /** sprite-ref: buttons; sprite-alignment: right */ + overflow: hidden; + text-overflow: ellipsis; } .#{$primaryStyleName}-caption { diff --git a/WebContent/VAADIN/themes/valo/components/_button.scss b/WebContent/VAADIN/themes/valo/components/_button.scss index 3a4bca615d..673a55a97a 100644 --- a/WebContent/VAADIN/themes/valo/components/_button.scss +++ b/WebContent/VAADIN/themes/valo/components/_button.scss @@ -119,6 +119,8 @@ text-align: center; white-space: nowrap; outline: none; + overflow: hidden; + text-overflow: ellipsis; @include valo-tappable; @if $vertical-centering { @include valo-button-vertical-centering; diff --git a/uitest/src/com/vaadin/tests/themes/ButtonTextOverflow.java b/uitest/src/com/vaadin/tests/themes/ButtonTextOverflow.java new file mode 100644 index 0000000000..6cf8b7958e --- /dev/null +++ b/uitest/src/com/vaadin/tests/themes/ButtonTextOverflow.java @@ -0,0 +1,27 @@ +package com.vaadin.tests.themes; + +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUI; +import com.vaadin.ui.Button; + +public class ButtonTextOverflow extends AbstractTestUI { + + @Override + protected void setup(VaadinRequest request) { + Button button = new Button("Button Button Button"); + + button.setWidth("100px"); + + addComponent(button); + } + + @Override + protected String getTestDescription() { + return "Overflowing button caption should be hidden with ellipsis."; + } + + @Override + protected Integer getTicketNumber() { + return 11864; + } +} diff --git a/uitest/src/com/vaadin/tests/themes/ButtonTextOverflowTest.java b/uitest/src/com/vaadin/tests/themes/ButtonTextOverflowTest.java new file mode 100644 index 0000000000..e31e98ac6e --- /dev/null +++ b/uitest/src/com/vaadin/tests/themes/ButtonTextOverflowTest.java @@ -0,0 +1,25 @@ +package com.vaadin.tests.themes; + +import java.io.IOException; + +import org.junit.Test; + +import com.vaadin.tests.tb3.MultiBrowserTest; + +public class ButtonTextOverflowTest extends MultiBrowserTest { + + @Test + public void captionOverflowIsHiddenInReindeer() throws IOException { + openTestURL("theme=reindeer"); + + compareScreen("reindeer"); + } + + @Test + public void captionOverflowIsHiddenInValo() throws IOException { + openTestURL("theme=valo"); + + compareScreen("valo"); + } + +} \ No newline at end of file -- 2.39.5