Fix wheel zoom

evgenyneu · Aug 6, 2020 · e6a1f42 · e6a1f42
1 parent 0812a76
commit e6a1f42
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/js/trajectories.js b/js/trajectories.js
@@ -6,7 +6,7 @@
  *
  * @return {object} Trajectory state object
  */
-export function init(positions, size=5000) {
+export function init(positions, size=10000) {
   let trajectories1 = Array(size * 3).fill(0);
   let trajectories2 = Array(size * 3).fill(0);
 

diff --git a/js/ui/zoom.js b/js/ui/zoom.js
@@ -9,9 +9,18 @@ function distanceBetweenFingers(e) {
 }
 
 
+
+/**
+ * User is rolling the mouse wheel, we need to zoom in/out
+ */
 function onWheel(state, e, currentParams) {
   e.preventDefault();
-  currentParams.cameraDistance += e.deltaY / 100 * currentParams.cameraDistance;
+
+  // The amount of wheel scroll, deltaY can vary between system.
+  // We want to normalise it by using the sign
+  const delta = Math.sign(e.deltaY);
+
+  currentParams.cameraDistance += delta / 5 * currentParams.cameraDistance;
 
   if (currentParams.cameraDistance > state.maxCameraDistance) {
     currentParams.cameraDistance = state.maxCameraDistance;