Ensure tap-dragging fires an update event

[Zirak]

Tap dragging did not fire a draw.update event, leading to much confusion.

In the interim, a fairly simple fix is to add the following before creating the relevant MapboxDraw instance:

MapboxDraw.modes.simple_select.onTouchEnd = MapboxDraw.modes.simple_select.onMouseUp;

[trygveaa]

I also ran into this issue. Can confirm that the fix is working properly. Can it be merged?

[Zirak]

I also ran into this issue. Can confirm that the fix is working properly. Can it be merged?

Until such a fateful day, you can workaround this bug as outlined in the PR message:

MapboxDraw.modes.simple_select.onTouchEnd = MapboxDraw.modes.simple_select.onMouseUp;

[trygveaa]

Thanks, yeah I know, but it's always better to have the fix upstream. And it would save time for the next person running into this.

[trygveaa]

@karimnaaji: Why was this closed? The issue still remains as far as I can see.

[karimnaaji]

Oups sorry about that @trygveaa ! We did a change of repository default branch yesterday and github closed all the PRs that were targetting the old branch as default target, this is the only PR we did not reopen.

[trygveaa]

Oh, so that's what happened. I did wonder why all my PRs were closed and reopened.

You have to change the destination too though. I see you have done it for most, but not all PRs.

[karimnaaji]

Yep working on it now :)

[stepankuzmin]

This was fixed before, but I will merge it to keep the unit test. Thanks for the contribution @Zirak!