From 08dbd4739c27bc6ac1eb3e5d655fbcfd3a8dfdd7 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Johannes=20Dahlstr=C3=B6m?= <johannesd@vaadin.com>
Date: Fri, 19 Oct 2012 17:42:10 +0300
Subject: [PATCH] Handle the situation where browser reports an unknown button
 on click event (#9956)

Change-Id: Ia6b1ebf42734d546cbba4e5e845d4c9998cdf9f7
---
 client/src/com/vaadin/client/MouseEventDetailsBuilder.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/client/src/com/vaadin/client/MouseEventDetailsBuilder.java b/client/src/com/vaadin/client/MouseEventDetailsBuilder.java
index cda2eec07f..85acf5000a 100644
--- a/client/src/com/vaadin/client/MouseEventDetailsBuilder.java
+++ b/client/src/com/vaadin/client/MouseEventDetailsBuilder.java
@@ -65,6 +65,9 @@ public class MouseEventDetailsBuilder {
             mouseEventDetails.setButton(MouseButton.RIGHT);
         } else if (evt.getButton() == NativeEvent.BUTTON_MIDDLE) {
             mouseEventDetails.setButton(MouseButton.MIDDLE);
+        } else {
+            // IE8 does not always report a button. Assume left.
+            mouseEventDetails.setButton(MouseButton.LEFT);
         }
         mouseEventDetails.setAltKey(evt.getAltKey());
         mouseEventDetails.setCtrlKey(evt.getCtrlKey());
-- 
2.39.5