chore: fix looped video preview seeking

justandras · justandras · commit d2f2fb0571c1 · 2026-06-18T11:05:36.000+02:00
diff --git a/packages/webui/src/client/lib/ui/videoPreviewScrub.ts b/packages/webui/src/client/lib/ui/videoPreviewScrub.ts
@@ -7,10 +7,10 @@ export function setVideoElementPosition(
 ): void {
 	let targetTime = timePosition + seek
 	if (loop && vEl.duration > 0) {
-		targetTime =
-			targetTime % ((itemDuration > 0 ? Math.min(vEl.duration * 1000, itemDuration) : vEl.duration * 1000) * 1000)
+		const loopWindowMs = itemDuration > 0 ? Math.min(vEl.duration * 1000, itemDuration) : vEl.duration * 1000
+		targetTime = ((targetTime % loopWindowMs) + loopWindowMs) % loopWindowMs
 	} else if (itemDuration > 0) {
-		targetTime = Math.min(targetTime, itemDuration)
+		targetTime = Math.max(0, Math.min(targetTime, itemDuration))
 	}
 	vEl.currentTime = targetTime / 1000
 }

Original file line number	Diff line number	Diff line change
`@@ -7,10 +7,10 @@ export function setVideoElementPosition(`
`7`	`7`	`): void {`
`8`	`8`	`let targetTime = timePosition + seek`
`9`	`9`	`if (loop && vEl.duration > 0) {`
`10`		`- targetTime =`
`11`		`- targetTime % ((itemDuration > 0 ? Math.min(vEl.duration * 1000, itemDuration) : vEl.duration * 1000) * 1000)`
	`10`	`+ const loopWindowMs = itemDuration > 0 ? Math.min(vEl.duration * 1000, itemDuration) : vEl.duration * 1000`
	`11`	`+ targetTime = ((targetTime % loopWindowMs) + loopWindowMs) % loopWindowMs`
`12`	`12`	`} else if (itemDuration > 0) {`
`13`		`- targetTime = Math.min(targetTime, itemDuration)`
	`13`	`+ targetTime = Math.max(0, Math.min(targetTime, itemDuration))`
`14`	`14`	`}`
`15`	`15`	`vEl.currentTime = targetTime / 1000`
`16`	`16`	`}`