Placed PointerLock inside pointermove event handler to ensure Safari compatibility for dragging

Author: seabeeberry

frontend/src/components/widgets/inputs/NumberInput.svelte

@@ -315,13 +315,6 @@
 		// Don't drag the text value from is input element
 		e.preventDefault();
 
-		// Get the event target and set the requestPointerLock immediately
-		// Safari seems to require requestPointerLock() to be called directly inside the pointerdown handler instead of later in beginDrag().
-		const target = e.target || undefined;
-		if (!(target instanceof HTMLElement)) return;
-
-		target.requestPointerLock();
-
 		// Now we need to wait and see if the user follows this up with a mousemove or mouseup.
 
 		// For some reason, both events can get fired before their event listeners are removed, so we need to guard against both running.
@@ -332,6 +325,13 @@
 			if (alreadyActedGuard) return;
 			alreadyActedGuard = true;
 			isDragging = true;
+
+			// We need to request pointer lock as immediately as possible for Safari compatibility
+			const target = e.target || undefined;
+			if (!(target instanceof HTMLElement)) return;
+
+			target.requestPointerLock();
+
 			beginDrag();
 			removeEventListener("pointermove", onMove);
 		};