fix: perform page moves only on drop, as this is an expensive process #…

…3126
bitfocus · Dec 15, 2024 · 24d98ad · 24d98ad
1 parent a2ab7fc
commit 24d98ad
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/webui/src/Buttons/Pages.tsx b/webui/src/Buttons/Pages.tsx
@@ -173,7 +173,8 @@ const PageListRow = observer(function PageListRow({
 	const ref = useRef<HTMLTableRowElement>(null)
 	const [, drop] = useDrop<PageListDragItem>({
 		accept: PAGE_LIST_DRAG_ID,
-		hover(item, _monitor) {
+		drop(item, _monitor) {
+			// We do this one on drop, as it is costly to move the page around
 			if (!ref.current) {
 				return
 			}