[BUG] Dynamic joysticks freezes on fast interaction

[caiiiycuk]

Hi.

Describe the bug
Dynamic joysticks freeze at some time if you interact fastly. This bug reproduced on nipple js website.

To Reproduce
Steps to reproduce the behavior:

1. Go to https://yoannmoi.net/nipplejs/
2. Fast interact with in dynamic mode
3. You should see some joysticks that are in freeze

Expected behavior
Joysticks should not freeze

Screenshots, Codepen or JSFiddle
Video: https://youtu.be/Rq1NxbSgD0I

Desktop (please complete the following information):

• Can't reproduce

Smartphone (please complete the following information):

• Device: [Lenovo Z6]; [Xiaomi Redmi 5A]; [Xiaomi Redmi 7A]; [Xiamoi Mi Max 2]
• OS: Android
• Chrome

[caiiiycuk]

I found that problem is exactly same as in #94, but for pointer events. Affected devices sometimes does not send pointerremove event. But seems that workaround as for touch events is not possible because we can't get ids of current pointers.

As workaround I switched to "touch" mode for default:

var toBind;
var secondBind = {};
if (isTouch) {
    toBind = events.touch;
    secondBind = events.mouse;
} else if (isPointer) {
    toBind = events.pointer;
} else if (isMSPointer) {
    toBind = events.MSPointer;
} else {
    toBind = events.mouse;
}

This helps. I think it could be good if this order can be configurable. Why pointer is preferred over touch?

[caiiiycuk]

I found that pointer events works much better then touch events. I added partial workaround for single touch mode it works fine in my case. Please look PR.

[binarybaba]

No activity here??

[yoannmoinet]

The PR that was fixing the issue has been closed due to not being robust enough unfortunately.

If you're willing to take over, feel free to do so.

[yoannmoinet]

I was able to reproduce on Chrome.
I'll have a look later today.
I'll also give some love to the lib, it's been drifting away for way too long.

[caiiiycuk]

@yoannmoinet I tried to fix it many times, I think there is no way. The problem is that android literally do not send some events, idk why, maybe due performance problems. I did some workarounds to reset the state in certain cases, but they are very ugly, not worst to show them)

[yoannmoinet]

I'll give it a try, see if I can think of something new.

As a last resort, I can just discard the touch after a static timeout, like in, if it didn't move in the last second, just trigger touchend on it.
It would probably fix 95% of the issues, but it might annoy 5% of the people that remained static on their touch for more than 1s 🤔

[yoannmoinet]

Can you show the workarounds you did @caiiiycuk, maybe it'll trigger some ideas as well.
Promise, I won't judge 😃

[caiiiycuk]

Can you show the workarounds you did @caiiiycuk, maybe it'll trigger some ideas as well. Promise, I won't judge 😃

This is what I ended with
master...caiiiycuk:nipplejs:gpx