apply Tomek suggestions

Expensify · Nov 26, 2024 · 954560c · 954560c
1 parent f631cb9
commit 954560c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/components/MultiGestureCanvas/usePanGesture.ts b/src/components/MultiGestureCanvas/usePanGesture.ts
@@ -204,7 +204,7 @@ const usePanGesture = ({
                 }
             }
 
-            if (previousTouchValue === null) {
+            if (previousTouch.get() === null) {
                 previousTouch.set({
                     x: evt.allTouches.at(0)?.x ?? 0,
                     y: evt.allTouches.at(0)?.y ?? 0,

diff --git a/src/components/MultiGestureCanvas/usePinchGesture.ts b/src/components/MultiGestureCanvas/usePinchGesture.ts
@@ -128,9 +128,8 @@ const usePinchGesture = ({
             }
 
             const newZoomScale = pinchScale.get() * evt.scale;
-            const zoomScaleValue = zoomScale.get();
             // Limit the zoom scale to zoom range including bounce range
-            if (zoomScaleValue >= zoomRange.min * ZOOM_RANGE_BOUNCE_FACTORS.min && zoomScaleValue <= zoomRange.max * ZOOM_RANGE_BOUNCE_FACTORS.max) {
+            if (zoomScale.get() >= zoomRange.min * ZOOM_RANGE_BOUNCE_FACTORS.min && zoomScale.get() <= zoomRange.max * ZOOM_RANGE_BOUNCE_FACTORS.max) {
                 zoomScale.set(newZoomScale);
                 currentPinchScale.set(evt.scale);
 
@@ -144,7 +143,7 @@ const usePinchGesture = ({
 
             // If the zoom scale is within the zoom range, we perform the regular pinch translation
             // Otherwise it means that we are "overzoomed" or "underzoomed", so we need to bounce back
-            if (zoomScaleValue >= zoomRange.min && zoomScaleValue <= zoomRange.max) {
+            if (zoomScale.get() >= zoomRange.min && zoomScale.get() <= zoomRange.max) {
                 pinchTranslateX.set(newPinchTranslateX);
                 pinchTranslateY.set(newPinchTranslateY);
             } else {