Changed VButton event handling so that isHovering is set to true onmousedown instead of onmouseover, fixes #7389

svn changeset:20310/svn branch:6.6

1 files changed, 6 insertions, 7 deletions

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VButton.java b/src/com/vaadin/terminal/gwt/client/ui/VButton.java
index 46492a707c..b0ef38733a 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VButton.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VButton.java
@@ -209,6 +209,12 @@ public class VButton extends FocusWidget implements Paintable, ClickHandler,
             }
             break;
         case Event.ONMOUSEDOWN:
+            if (DOM.isOrHasChild(getElement(), DOM.eventGetTarget(event))) {
+                // This was moved from mouseover, which iOS sometimes skips.
+                // We're certainly hovering at this point, and we don't actually
+                // need that information before this point.
+                setHovering(true);
+            }
             if (event.getButton() == Event.BUTTON_LEFT) {
                 // save mouse position to detect movement before synthesizing
                 // event later
@@ -266,13 +272,6 @@ public class VButton extends FocusWidget implements Paintable, ClickHandler,
                 }
             }
             break;
-        case Event.ONMOUSEOVER:
-            if (DOM.isOrHasChild(getElement(), DOM.eventGetTarget(event))) {
-                setHovering(true);
-                if (isCapturing) {
-                }
-            }
-            break;
         case Event.ONBLUR:
             if (isFocusing) {
                 isFocusing = false;