fix: 🐛 inaccurate calculation in target scale (#367)

imtrent · web-flow · commit e8117f61cf25 · 2023-04-11T18:42:02.000+02:00
diff --git a/src/core/handlers/handlers.utils.ts b/src/core/handlers/handlers.utils.ts
@@ -24,7 +24,7 @@ export const handleCalculateButtonZoom = (
     throw new Error("Wrapper is not mounted");
   }
 
-  const targetScale = scale * Math.exp(delta * step);
+  const targetScale = scale + delta * step;
 
   const newScale = checkZoomBounds(
     roundNumber(targetScale, 3),
diff --git a/src/core/wheel/wheel.utils.ts b/src/core/wheel/wheel.utils.ts
@@ -86,7 +86,7 @@ export const handleCalculateWheelZoom = (
     throw new Error("Wrapper is not mounted");
   }
 
-  const targetScale = scale + delta * (scale - scale * step) * step;
+  const targetScale = scale + delta * step;
 
   if (getTarget) return targetScale;
   const paddingEnabled = disable ? false : !disabled;

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ export const handleCalculateButtonZoom = (`
`24`	`24`	`throw new Error("Wrapper is not mounted");`
`25`	`25`	`}`
`26`	`26`
`27`		`- const targetScale = scale * Math.exp(delta * step);`
	`27`	`+ const targetScale = scale + delta * step;`
`28`	`28`
`29`	`29`	`const newScale = checkZoomBounds(`
`30`	`30`	`roundNumber(targetScale, 3),`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ export const handleCalculateWheelZoom = (`
`86`	`86`	`throw new Error("Wrapper is not mounted");`
`87`	`87`	`}`
`88`	`88`
`89`		`- const targetScale = scale + delta * (scale - scale * step) * step;`
	`89`	`+ const targetScale = scale + delta * step;`
`90`	`90`
`91`	`91`	`if (getTarget) return targetScale;`
`92`	`92`	`const paddingEnabled = disable ? false : !disabled;`