diff options
| author | Alexey Fansky <alexey.fansky@effective-soft.com> | 2015-04-09 11:09:15 -0700 |
|---|---|---|
| committer | Vaadin Code Review <review@vaadin.com> | 2015-04-10 12:23:10 +0000 |
| commit | 817906332a8ba99e8a711067c10282b8cd9a6415 (patch) | |
| tree | 6ddac9fcf62c73c584bea4c99aec5a395edcd587 /client/src | |
| parent | 4217d7bbc8552e822367fac9d7d3cc9ee48821dd (diff) | |
| download | vaadin-framework-817906332a8ba99e8a711067c10282b8cd9a6415.tar.gz vaadin-framework-817906332a8ba99e8a711067c10282b8cd9a6415.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() { |