diff options
| author | Alexey Fansky <alexey.fansky@effective-soft.com> | 2015-04-09 11:09:15 -0700 |
|---|---|---|
| committer | Markus Koivisto <markus@vaadin.com> | 2015-04-15 11:53:08 +0300 |
| commit | 583750936967e2eb2b35d48a8ea2519f351db1de (patch) | |
| tree | c3a688f2bcbf19b76c52b3c94e377f6ca6ca3e82 /client/src | |
| parent | 3ac8bf71e4b88e4c0fe683c84940509d6194c782 (diff) | |
| download | vaadin-framework-583750936967e2eb2b35d48a8ea2519f351db1de.tar.gz vaadin-framework-583750936967e2eb2b35d48a8ea2519f351db1de.zip | |
Adding a temporary style to VNotification on Chrome >=41 (#17252)
Change-Id: I41c05e8f8487d351035804e0681947956d861479
Diffstat (limited to 'client/src')
| -rw-r--r-- | client/src/com/vaadin/client/ui/VNotification.java | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/client/src/com/vaadin/client/ui/VNotification.java b/client/src/com/vaadin/client/ui/VNotification.java index d7639b0022..eee0f459a6 100644 --- a/client/src/com/vaadin/client/ui/VNotification.java +++ b/client/src/com/vaadin/client/ui/VNotification.java @@ -257,12 +257,19 @@ public class VNotification extends VOverlay { /** * Android 4 fails to render notifications correctly without a little * nudge (#8551) + * Chrome 41 now requires this too (#17252) */ - if (BrowserInfo.get().isAndroid()) { + if (BrowserInfo.get().isAndroid() + || isChrome41OrHigher()) { WidgetUtil.setStyleTemporarily(getElement(), "display", "none"); } } + private boolean isChrome41OrHigher() { + return BrowserInfo.get().isChrome() + && BrowserInfo.get().getBrowserMajorVersion() >= 41; + } + protected void hideAfterDelay() { if (delay == null) { delay = new Timer() { |