-
-
Notifications
You must be signed in to change notification settings - Fork 291
Impossible to produce wheel actions on a trackpad #617
Description
Describe the bug
Hello, I am using camera-controls with perspective camera and mouse wheel action set to dolly. Since merging of #607 it is now impossible to dolly camera on trackpad since the library overrides the configuration to always zoom.
I don't think the rationale in #604 is a cause for changing the library. What was described is expected behavior (action configured to truck - camera trucks). Trackpad produces mouse wheel event for a reason, and it is to adhere to web standards. If somebody wants to implement pinch to zoom it should be done in client space and not in the library internals (like shown in https://github.com/yomotsu/camera-controls/blob/dev/examples/mouse-drag-with-modifier-keys.html). Overriding user config in library is a violation of an API contract and also it shouldn't have been done in patch version bump. Please revert this change.
To Reproduce
Steps to reproduce the behavior:
- Set the mouse wheel action to dolly
- Pinch on a trackpad
- Observe camera zoom instead of dolly
Code
Live example
No response
Expected behavior
Camera dollies in/out
Screenshots or Video
No response
Device
Desktop
OS
MacOS
Browser
Chrome