From: Automerge <automerge@vaadin.com>
Date: Mon, 5 Mar 2012 10:05:36 +0000 (+0000)
Subject: [merge from 6.7] Fixed #6753: IE 6 to 8 propagate button clicks to layout/window
X-Git-Tag: 7.0.0.alpha2~240^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0b46687ab0cedb3523321854b52d988dae18452a;p=vaadin-framework.git

[merge from 6.7] Fixed #6753: IE 6 to 8 propagate button clicks to layout/window

svn changeset:23176/svn branch:6.8
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VButton.java b/src/com/vaadin/terminal/gwt/client/ui/VButton.java
index 03de99ada6..9188f7406a 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VButton.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VButton.java
@@ -250,6 +250,12 @@ public class VButton extends FocusWidget implements Paintable, ClickHandler,
                 if (BrowserInfo.get().isIE() || BrowserInfo.get().isOpera()) {
                     removeStyleName(CLASSNAME_PRESSED);
                 }
+                // Explicitly prevent IE 6 to 8 from propagating mouseup events
+                // upward (fixes #6753)
+                if (BrowserInfo.get().isIE()
+                        && BrowserInfo.get().getIEVersion() < 9) {
+                    event.stopPropagation();
+                }
             }
             break;
         case Event.ONMOUSEMOVE: