From 74612882faf12886352eee0f5fb51839725a369d Mon Sep 17 00:00:00 2001 From: Matthew Somerville Date: Fri, 27 Sep 2024 18:38:08 +0100 Subject: [PATCH] Fix touchmove logic in buttonclick. The incorrect bracket placement meant that touch events were not being cancelled should you move your finger, meaning e.g. a zoom would still occur if you touched a zoom, moved your finger off, then lifted. --- lib/OpenLayers/Events/buttonclick.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/OpenLayers/Events/buttonclick.js b/lib/OpenLayers/Events/buttonclick.js index 6879e60f50..7f02ffc4ee 100644 --- a/lib/OpenLayers/Events/buttonclick.js +++ b/lib/OpenLayers/Events/buttonclick.js @@ -198,7 +198,7 @@ OpenLayers.Events.buttonclick = OpenLayers.Class({ if (this.cancelRegEx.test(evt.type)) { if (evt.touches && this.startEvt.touches && (Math.abs(evt.touches[0].olClientX - this.startEvt.touches[0].olClientX) > 4 || - Math.abs(evt.touches[0].olClientY - this.startEvt.touches[0].olClientY)) > 4) { + Math.abs(evt.touches[0].olClientY - this.startEvt.touches[0].olClientY) > 4)) { delete this.startEvt; } }