From: Artur Signell <artur.signell@itmill.com>
Date: Fri, 6 Feb 2009 08:42:13 +0000 (+0000)
Subject: Fix for #2501 - Changed notification event handling so it only reacts to the events... 
X-Git-Tag: 6.7.0.beta1~3205
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=eb6e305de07f954746d896b251bbc0c6f142535c;p=vaadin-framework.git

Fix for #2501 - Changed notification event handling so it only reacts to the events we know that should fade the notification

svn changeset:6746/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/INotification.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/INotification.java
index 3f4aea0c64..1186f99ad5 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/INotification.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/INotification.java
@@ -260,10 +260,19 @@ public class INotification extends IToolkitOverlay {
                 startDelay();
             }
             break;
-        default:
-            if (type != Event.ONMOUSEUP && type != Event.ONKEYUP) {
-                startDelay();
+        case Event.ONMOUSEDOWN:
+        case Event.ONMOUSEWHEEL:
+        case Event.ONSCROLL:
+            startDelay();
+            break;
+        case Event.ONKEYDOWN:
+            if (event.getRepeat()) {
+                return true;
             }
+            startDelay();
+            break;
+        default:
+            break;
         }
         return true;
     }