Fix mouse functionality on touch devices.

[danrha]

Description

Remove special case for touch devices when adding event listeners to the view.

Related issues

• This fixes Clicks not detected when connecting a mouse to iPad  #1959

[eeropic]

Hmm, looking at how it's implemented at the moment

paper.js/src/view/View.js

Line 1073 in 3ef3ca6

if (navigator.pointerEnabled || navigator.msPointerEnabled) {

It is checking
if (navigator.pointerEnabled || navigator.msPointerEnabled) {

But it should be checking if ('PointerEvent' in window) { instead.

Could you try this instead of the changes you are proposing, and check if it solves it,
if so - then it would be good to go from my point of view. I tried this when I had issues with Apple Pencil and touch simultaneously.

[eeropic]

You should try changing line 1073

paper.js/src/view/View.js

Line 1073 in 3ef3ca6

if (navigator.pointerEnabled || navigator.msPointerEnabled) {

to if ('PointerEvent' in window) { and check whether it solves your problem

(instead of the changes proposed below)

[eeropic]

Unfortunately there seems to be some possible breaking behaviour, which I'm not fully aware of atm #1810 (comment)

[danrha]

Using PointerEvent does fix it for me but I still see no harm in always listening to all the events instead of depending on the userAgent string. The mouse events shouldn't be called after calling preventDefault on the touch events anyways.