context menu issue fixed

[terrytilley]

Issue

When viewing the slider on mobile and performing a long press on the image opens the context menu, once you exit the menu endtouchHandler() is never called. You are no longer able to swipe left or right because touch.count is equal to 2 and touch.multitouch is equal to true.

Fix

If the the context menu has been activated it will then call the touchendHandler() function.

[feimosi]

Thanks for the contribution!
Firstly, could you remove the package-lock.json file from the PR? I'll add it later myself.
Secondly (if you have some time), could you move this logic to a separate method (named contextmenuHandler and attach it in the bindEvents method? Just like other handlers.

[feimosi]

Great, we're almost there. bind attaches and detaches the event listener automatically and there's also slider available in the scope, so you can just write:

var contextmenuHandler = function() {
    touchendHandler();
};

bind(slider, 'contextmenu', contextmenuHandler);

Just test it, to be sure, thanks!

[terrytilley]

That's is a much better solution 👍

All seems to be working so hopefully that is now fixed.

[feimosi]

Great, then I'm merging, thanks!