From 7e06c8526bef0e904a5cb5cfa97573bb753092ec Mon Sep 17 00:00:00 2001
From: Jean-Luc Makiola <business@jeanlucmakiola.de>
Date: Mon, 16 Mar 2026 22:36:40 +0100
Subject: [PATCH] fix(11): wire handleDragEnd to Reorder.Group for active
 threads

onPointerUp was incorrectly placed on the resolved-thread div instead
of the active-thread Reorder.Group, causing drag reorder to not persist.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/client/routes/threads/$threadId.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/client/routes/threads/$threadId.tsx b/src/client/routes/threads/$threadId.tsx
index 72c936a..f16ccc2 100644
--- a/src/client/routes/threads/$threadId.tsx
+++ b/src/client/routes/threads/$threadId.tsx
@@ -195,6 +195,7 @@ function ThreadDetailPage() {
 						axis="y"
 						values={displayItems}
 						onReorder={setTempItems}
+						onPointerUp={handleDragEnd}
 						className="flex flex-col gap-2"
 					>
 						{displayItems.map((candidate, index) => (
@@ -213,7 +214,7 @@ function ThreadDetailPage() {
 						))}
 					</Reorder.Group>
 				) : (
-					<div className="flex flex-col gap-2" onPointerUp={handleDragEnd}>
+					<div className="flex flex-col gap-2">
 						{displayItems.map((candidate, index) => (
 							<CandidateListItem
 								key={candidate.id}