OrbitControls: Implement Pointer Events.

[mrdoob]

When using mousedown, mousemove and mouseup, event.clientX and event.clientY are integers. In HiDPI (Retina) displays sometimes the cursor moves but the camera doesn't.

When using pointerdown, pointermove and pointerup (supported even in IE11) these properties are floats and manipulating the scene is snappier.

You can try the change by dragging the scene very slowly:

http://raw.githack.com/mrdoob/three.js/r119/examples/webgl_animation_cloth.html
http://raw.githack.com/mrdoob/three.js/dev/examples/webgl_animation_cloth.html

@Mugen87 We should definitely implement this in all relevant controls.

We could also use Pointer Events for touch too but we currently rely in event.touches which Pointer Events don't have.

[mrdoob]

/cc @WestLangley

[Mugen87]

Related issue: #4015

The difference is really noticeable 😳.

We should definitely implement this in all relevant controls.

Agreed! It should be DragControls, FirstPersonControls, EditorControls, FlyControls, PointerLockControls, TrackballControls and TransformControls.

[Mugen87]

I've noticed right now that the event data movementX and movementY are still integers when using pointermove. Should we still introduce Pointer Events to PointerLockControls?

[mrdoob]

movementX and movementY do not seem to work on Safari 😕

At least it the Resizer code doesn't seem to work.

[Mugen87]

Yeah, I see it now too. PointerLockControls would be broken in Safari after this change. Okay, let's leave this class out for now.

[marcofugaro]

Nice! I like this improvement.

We could also use Pointer Events for touch too but we currently rely in event.touches which Pointer Events don't have.

For Pointer Events the pattern is different, you'd have to keep track yourself of how many pointerdown events happened, say with a counter that increases when a pointerdown event happens and decreases on pointerup/pointerleave.

Read more about it here: https://stackoverflow.com/questions/50955861/detect-two-finger-touch-with-pointerevents-on-the-same-target