use-gesture: Exception thrown when holding ctrl and simultaneously scroll the wheel on the mouse fast enough
Describe the bug
Exception thrown when holding ctrl and simultaneously scroll the wheel on the mouse fast enough.
TypeError
Cannot read properties of undefined (reading 'clientX')
at distanceAngle (https://7rfcn.csb.app/node_modules/
use-gesture/core/dist/actions-71ad3053.esm.js:242:17
at PinchEngine.pointerMove (https://7rfcn.csb.app/node_modules/
use-gesture/core/dist/actions-71ad3053.esm.js:1157:21
at HTMLDivElement.eval (https://7rfcn.csb.app/node_modules/
use-gesture/core/dist/actions-71ad3053.esm.js:316:19
Sandbox or Video Card-zoom sandbox from the documentation: https://codesandbox.io/s/github/pmndrs/use-gesture/tree/main/demo/src/sandboxes/card-zoom
Information:
- Use Gesture version: 10.2.21
- Device: Laptop
- OS: Windows 11
- Browser Chrome
Checklist:
- I’ve read the documentation.
- If this is an issue with drag, I’ve tried setting
touch-action: noneto the draggable element.
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 22
Released 10.2.26!
Indeed, the error is gone. Thank you for fixing 😃
@dbismut The error is gone 🙂I can easily reproduce an error on all previous versions, but not on
pmndrs/use-gesture#pull/596/head.@baptisteArno I’ve just tried Browserstack and the problem doesn’t show when using my MagicMouse. I’ll try fixing this with my eyes closed.
@dbismut you’re welcome.
The error must be something that takes a while like caching pointer ids or not forgetting stale pointer ids or storing pointer ids in internal storage or calculating configuration at setup and not updating it.
I tried everything to find a clear way of reproducing the error and I am sorry but ‘after a while’ is all I got.
Also, I think you should make multi hands gesture an opt-in feature (regardless of the error) I think, by default, people (and dumb people like me) don’t want it (at least for simple use cases).