Skip to content

Fix 'simple_select' mode lacking 'touchend' event handling (#684)#1008

Merged
mourner merged 2 commits intomapbox:mainfrom
corinv:issue-684
May 19, 2021
Merged

Fix 'simple_select' mode lacking 'touchend' event handling (#684)#1008
mourner merged 2 commits intomapbox:mainfrom
corinv:issue-684

Conversation

@corinv
Copy link
Contributor

@corinv corinv commented Aug 27, 2020

Fixes (#684).

The draw.update event would not be fired on moving a geometry using a touch device (or emulated touch, like Chrome's device mode) as there was no handling for a touchend event to fire said event in the simple_select mode.

@corinv corinv changed the title Fix 'simple_select' mode lacking 'touchend' event (#684) Fix 'simple_select' mode lacking 'touchend' event handling (#684) Aug 27, 2020
@gbuckingham89
Copy link

Would love to see this merged - it seems to fix an issue we're having on touch devices (iPad) where dragging polygon in simple_touch mode never ends until a tap is made (on the polygon or the map) - meaning you can't touch and rag the polygon, then immediately touch to pan the map.

@mourner mourner merged commit ce122fb into mapbox:main May 19, 2021
@mourner
Copy link
Member

mourner commented May 19, 2021

Sorry for getting back so late on this, and thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants