Normalize scroll zoom sensitivity for mouse vs touchpad

wenli-cai · wenli-cai · commit c6264da42314 · 2025-07-10T10:31:33.000-04:00
diff --git a/workflow-trace-viewer/src/jvmMain/kotlin/com/squareup/workflow1/traceviewer/util/SandboxBackground.kt b/workflow-trace-viewer/src/jvmMain/kotlin/com/squareup/workflow1/traceviewer/util/SandboxBackground.kt
@@ -48,8 +48,8 @@ public fun SandboxBackground(
           val event = awaitPointerEvent()
           if (event.type == PointerEventType.Scroll) {
             val scrollDelta = event.changes.first().scrollDelta.y
-            sandboxState.scale = (sandboxState.scale * if (scrollDelta < 0) 1.1f else 0.9f)
-              .coerceIn(0.1f, 10f)
+            val factor = 1f + (-scrollDelta * 0.1f)
+            sandboxState.scale = (sandboxState.scale * factor).coerceIn(0.1f, 10f)
             event.changes.forEach { it.consume() }
           }
         }

Original file line number	Diff line number	Diff line change
`@@ -48,8 +48,8 @@ public fun SandboxBackground(`
`48`	`48`	`val event = awaitPointerEvent()`
`49`	`49`	`if (event.type == PointerEventType.Scroll) {`
`50`	`50`	`val scrollDelta = event.changes.first().scrollDelta.y`
`51`		`- sandboxState.scale = (sandboxState.scale * if (scrollDelta < 0) 1.1f else 0.9f)`
`52`		`- .coerceIn(0.1f, 10f)`
	`51`	`+ val factor = 1f + (-scrollDelta * 0.1f)`
	`52`	`+ sandboxState.scale = (sandboxState.scale * factor).coerceIn(0.1f, 10f)`
`53`	`53`	`event.changes.forEach { it.consume() }`
`54`	`54`	`}`
`55`	`55`	`}`