From 77766682691e18f6e8bc7fbaee6f12cff2c9e9a5 Mon Sep 17 00:00:00 2001 From: Hermes Date: Sat, 1 Aug 2026 13:09:35 +0000 Subject: [PATCH] fix: remove all state changes on note selection to prevent focus stealing --- apps/web/src/routes/_app/notes.tsx | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/apps/web/src/routes/_app/notes.tsx b/apps/web/src/routes/_app/notes.tsx index 80991a8..bd4e79a 100644 --- a/apps/web/src/routes/_app/notes.tsx +++ b/apps/web/src/routes/_app/notes.tsx @@ -80,7 +80,7 @@ const NoteTitleInput = memo(function NoteTitleInput({ noteId, initialTitle }: { function NotesPage() { const queryClient = useQueryClient(); const [search, setSearch] = useState(""); - const [selectedNoteId, setSelectedNoteId] = useState(null); + const selectedNoteIdRef = useRef(null); const [showBacklinks, setShowBacklinks] = useState(false); const [showVersions, setShowVersions] = useState(false); const selectedNoteRef = useRef(null); @@ -99,7 +99,7 @@ function NotesPage() { onSuccess: (note) => { queryClient.invalidateQueries({ queryKey: ["notes"] }); selectedNoteRef.current = note; - setSelectedNoteId(note.id); + selectedNoteIdRef.current = note.id; }, }); @@ -113,7 +113,7 @@ function NotesPage() { onSuccess: () => { queryClient.invalidateQueries({ queryKey: ["notes"] }); selectedNoteRef.current = null; - setSelectedNoteId(null); + selectedNoteIdRef.current = null; }, }); @@ -124,7 +124,7 @@ function NotesPage() { } catch { selectedNoteRef.current = note; } - setSelectedNoteId(note.id); + selectedNoteIdRef.current = note.id; setShowBacklinks(false); setShowVersions(false); }; @@ -159,7 +159,7 @@ function NotesPage() { onClick={() => selectNote(note)} className={cn( "w-full text-left px-3 py-2 rounded-md text-sm transition-colors", - selectedNoteId === note.id ? "bg-accent text-accent-foreground" : "hover:bg-accent/50" + selectedNoteIdRef.current === note.id ? "bg-accent text-accent-foreground" : "hover:bg-accent/50" )} >