fix: save note title on blur instead of every keystroke to prevent focus leak
This commit is contained in:
@@ -53,10 +53,10 @@ function NotesPage() {
|
|||||||
const queryClient = useQueryClient();
|
const queryClient = useQueryClient();
|
||||||
const [search, setSearch] = useState("");
|
const [search, setSearch] = useState("");
|
||||||
const [selectedNote, setSelectedNote] = useState<Note | null>(null);
|
const [selectedNote, setSelectedNote] = useState<Note | null>(null);
|
||||||
|
const [titleDraft, setTitleDraft] = useState("");
|
||||||
const [editorContent, setEditorContent] = useState("");
|
const [editorContent, setEditorContent] = useState("");
|
||||||
const saveTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
const saveTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
||||||
const titleTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
const titleTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
||||||
const titleTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
|
||||||
const [showBacklinks, setShowBacklinks] = useState(false);
|
const [showBacklinks, setShowBacklinks] = useState(false);
|
||||||
const [showVersions, setShowVersions] = useState(false);
|
const [showVersions, setShowVersions] = useState(false);
|
||||||
|
|
||||||
@@ -95,9 +95,11 @@ function NotesPage() {
|
|||||||
try {
|
try {
|
||||||
const detail = await api.get<Note>("/notes/" + note.id);
|
const detail = await api.get<Note>("/notes/" + note.id);
|
||||||
setSelectedNote(detail);
|
setSelectedNote(detail);
|
||||||
|
setTitleDraft(detail.title);
|
||||||
setEditorContent(detail.content || "");
|
setEditorContent(detail.content || "");
|
||||||
} catch {
|
} catch {
|
||||||
setSelectedNote(note);
|
setSelectedNote(note);
|
||||||
|
setTitleDraft(note.title);
|
||||||
setEditorContent(note.content || "");
|
setEditorContent(note.content || "");
|
||||||
}
|
}
|
||||||
setShowBacklinks(false);
|
setShowBacklinks(false);
|
||||||
@@ -115,12 +117,13 @@ function NotesPage() {
|
|||||||
}, [selectedNote, updateMutation]);
|
}, [selectedNote, updateMutation]);
|
||||||
|
|
||||||
const handleTitleChange = (title: string) => {
|
const handleTitleChange = (title: string) => {
|
||||||
if (selectedNote) {
|
setTitleDraft(title);
|
||||||
setSelectedNote({ ...selectedNote, title });
|
};
|
||||||
if (titleTimerRef.current) clearTimeout(titleTimerRef.current);
|
|
||||||
titleTimerRef.current = setTimeout(() => {
|
const handleTitleBlur = () => {
|
||||||
updateMutation.mutate({ id: selectedNote.id, data: { title } });
|
if (selectedNote && titleDraft !== selectedNote.title) {
|
||||||
}, 500);
|
updateMutation.mutate({ id: selectedNote.id, data: { title: titleDraft } });
|
||||||
|
setSelectedNote({ ...selectedNote, title: titleDraft });
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -175,8 +178,9 @@ function NotesPage() {
|
|||||||
<>
|
<>
|
||||||
<div className="flex items-center gap-2 p-3 border-b">
|
<div className="flex items-center gap-2 p-3 border-b">
|
||||||
<Input
|
<Input
|
||||||
value={selectedNote.title}
|
value={titleDraft}
|
||||||
onChange={(e) => handleTitleChange(e.target.value)}
|
onChange={(e) => handleTitleChange(e.target.value)}
|
||||||
|
onBlur={handleTitleBlur}
|
||||||
className="text-lg font-semibold border-0 focus-visible:ring-0 px-0"
|
className="text-lg font-semibold border-0 focus-visible:ring-0 px-0"
|
||||||
/>
|
/>
|
||||||
<div className="flex items-center gap-1 shrink-0">
|
<div className="flex items-center gap-1 shrink-0">
|
||||||
|
|||||||
Reference in New Issue
Block a user