From 20be404472b2310a7a05b29a7af2f2e8bfe63974 Mon Sep 17 00:00:00 2001
From: Jon Funkhouser <jonathanf@ladodgers.com>
Date: Tue, 11 Sep 2018 12:18:17 -0700
Subject: [PATCH 1/3] touch events: initialize clientX and clientY

this sets clientX and clientY to the values from the first touch in TouchEvent.changedTouches
---
 src/components/dragelement/index.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/components/dragelement/index.js b/src/components/dragelement/index.js
index 7f2a1adae6d..9d9443f31d0 100644
--- a/src/components/dragelement/index.js
+++ b/src/components/dragelement/index.js
@@ -137,6 +137,12 @@ dragElement.init = function init(options) {
         initialEvent = e;
         rightClick = e.buttons === 2 || e.ctrlKey;
 
+        // fix Fx.hover for touch events
+        if(typeof e.clientX === 'undefined' && typeof e.clientY === 'undefined') {
+            e.clientX = startX;
+            e.clientY = startY;
+        }
+
         newMouseDownTime = (new Date()).getTime();
         if(newMouseDownTime - gd._mouseDownTime < DBLCLICKDELAY) {
             // in a click train

From dddbef4850b1fdc90701b616d3ed5cd8bd039170 Mon Sep 17 00:00:00 2001
From: Jon Funkhouser <jonathanf@ladodgers.com>
Date: Mon, 17 Sep 2018 16:27:43 -0700
Subject: [PATCH 2/3] add test for click events on touch devices

---
 test/jasmine/tests/hover_label_test.js | 53 ++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

diff --git a/test/jasmine/tests/hover_label_test.js b/test/jasmine/tests/hover_label_test.js
index 437c47392d1..30b09a10eb5 100644
--- a/test/jasmine/tests/hover_label_test.js
+++ b/test/jasmine/tests/hover_label_test.js
@@ -13,6 +13,7 @@ var click = require('../assets/click');
 var delay = require('../assets/delay');
 var doubleClick = require('../assets/double_click');
 var failTest = require('../assets/fail_test');
+var touchEvent = require('../assets/touch_event');
 
 var customAssertions = require('../assets/custom_assertions');
 var assertHoverLabelStyle = customAssertions.assertHoverLabelStyle;
@@ -21,6 +22,20 @@ var assertElemRightTo = customAssertions.assertElemRightTo;
 var assertElemTopsAligned = customAssertions.assertElemTopsAligned;
 var assertElemInside = customAssertions.assertElemInside;
 
+function touch(path, options) {
+    var len = path.length;
+    Lib.clearThrottle();
+    touchEvent('touchstart', path[0][0], path[0][1], options);
+
+    path.slice(1, len).forEach(function(pt) {
+        Lib.clearThrottle();
+        touchEvent('touchmove', pt[0], pt[1], options);
+    });
+
+    touchEvent('touchend', path[len - 1][0], path[len - 1][1], options);
+    return;
+}
+
 describe('hover info', function() {
     'use strict';
 
@@ -2503,3 +2518,41 @@ describe('hovermode defaults to', function() {
           .then(done);
     });
 });
+
+
+describe('touch devices', function() {
+    ['pan', 'zoom'].forEach(function(type) {
+        describe('dragmode:' + type, function() {
+            var data = [{x: [1, 2, 3], y: [1, 3, 2], type: 'bar'}];
+            var layout = {width: 600, height: 400, dragmode: type};
+            var gd;
+
+            beforeEach(function(done) {
+                gd = createGraphDiv();
+                Plotly.plot(gd, data, layout).then(done);
+            });
+
+            it('emits click events', function(done) {
+                var hoverHandler = jasmine.createSpy('hover');
+                var clickHandler = jasmine.createSpy('click');
+                gd.on('plotly_hover', hoverHandler);
+                gd.on('plotly_click', clickHandler);
+
+                var gdBB = gd.getBoundingClientRect();
+                var touchPoint = [[gdBB.left + 300, gdBB.top + 200]];
+
+                Promise.resolve()
+                    .then(function() {
+                        touch(touchPoint);
+                    })
+                    .then(delay(HOVERMINTIME * 1.1))
+                    .then(function() {
+                        expect(clickHandler).toHaveBeenCalled();
+                        expect(hoverHandler).not.toHaveBeenCalled();
+                    })
+                    .catch(failTest)
+                    .then(done);
+            });
+        });
+    });
+});

From fca6f9b115a7b9f2eb35b0f32f0e263865754b3a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89tienne=20T=C3=A9treault-Pinard?= <etienne@plot.ly>
Date: Tue, 18 Sep 2018 10:40:18 -0400
Subject: [PATCH 3/3] cleanup after new touch hover test

---
 test/jasmine/tests/hover_label_test.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/jasmine/tests/hover_label_test.js b/test/jasmine/tests/hover_label_test.js
index 30b09a10eb5..9469a876f37 100644
--- a/test/jasmine/tests/hover_label_test.js
+++ b/test/jasmine/tests/hover_label_test.js
@@ -2521,6 +2521,8 @@ describe('hovermode defaults to', function() {
 
 
 describe('touch devices', function() {
+    afterEach(destroyGraphDiv);
+
     ['pan', 'zoom'].forEach(function(type) {
         describe('dragmode:' + type, function() {
             var data = [{x: [1, 2, 3], y: [1, 3, 2], type: 'bar'}];