From 041619821ff01784a07b6d2a2acf233e2bf9aa2f Mon Sep 17 00:00:00 2001
From: PJ <ipriyanshujain@gmail.com>
Date: Sat, 20 Jun 2026 18:12:15 -0400
Subject: [PATCH] feat(settings): draft Book setup, commit changes only on Save

---
 src/components/Settings.tsx | 50 ++++++++++++++++++++++++++-----------
 1 file changed, 36 insertions(+), 14 deletions(-)

diff --git a/src/components/Settings.tsx b/src/components/Settings.tsx
index 6900d19..049ba73 100644
--- a/src/components/Settings.tsx
+++ b/src/components/Settings.tsx
@@ -1,3 +1,4 @@
+import { useState } from "react";
 import type { ReactNode } from "react";
 import { useBook } from "../store/useBook";
 import { Icon } from "./Icon";
@@ -11,6 +12,15 @@ const LANGUAGES = [
   { value: "pt", label: "Portuguese" },
 ];
 
+interface Draft {
+  title: string;
+  subtitle: string;
+  author: string;
+  isbn: string;
+  language: string;
+  bleed: boolean;
+}
+
 function Field({ label, children }: { label: string; children: ReactNode }) {
   return (
     <label className="field">
@@ -25,8 +35,26 @@ export function Settings({ onClose, onSave }: { onClose: () => void; onSave: ()
   const setMetadata = useBook((s) => s.setMetadata);
   const setSettings = useBook((s) => s.setSettings);
 
+  const [draft, setDraft] = useState<Draft>(() => ({
+    title: book?.metadata.title ?? "",
+    subtitle: book?.metadata.subtitle ?? "",
+    author: book?.metadata.author ?? "",
+    isbn: book?.metadata.isbn ?? "",
+    language: book?.metadata.language ?? "en",
+    bleed: book?.settings.bleed ?? true,
+  }));
+
   if (!book) return null;
-  const { metadata, settings } = book;
+
+  const set = (patch: Partial<Draft>) => setDraft((d) => ({ ...d, ...patch }));
+
+  const save = () => {
+    const { bleed, ...metadata } = draft;
+    setMetadata(metadata);
+    setSettings({ bleed });
+    onSave();
+    onClose();
+  };
 
   return (
     <div className="overlay" onClick={onClose}>
@@ -39,19 +67,19 @@ export function Settings({ onClose, onSave }: { onClose: () => void; onSave: ()
         </div>
         <div className="panel-body">
           <Field label="Title">
-            <input value={metadata.title} placeholder="Untitled" onChange={(e) => setMetadata({ title: e.target.value })} />
+            <input value={draft.title} placeholder="Untitled" onChange={(e) => set({ title: e.target.value })} />
           </Field>
           <Field label="Subtitle">
-            <input value={metadata.subtitle} onChange={(e) => setMetadata({ subtitle: e.target.value })} />
+            <input value={draft.subtitle} onChange={(e) => set({ subtitle: e.target.value })} />
           </Field>
           <Field label="Author">
-            <input value={metadata.author} onChange={(e) => setMetadata({ author: e.target.value })} />
+            <input value={draft.author} onChange={(e) => set({ author: e.target.value })} />
           </Field>
           <Field label="ISBN">
-            <input value={metadata.isbn} placeholder="optional" onChange={(e) => setMetadata({ isbn: e.target.value })} />
+            <input value={draft.isbn} placeholder="optional" onChange={(e) => set({ isbn: e.target.value })} />
           </Field>
           <Field label="Language">
-            <select value={metadata.language} onChange={(e) => setMetadata({ language: e.target.value })}>
+            <select value={draft.language} onChange={(e) => set({ language: e.target.value })}>
               {LANGUAGES.map((l) => (
                 <option key={l.value} value={l.value}>
                   {l.label}
@@ -60,18 +88,12 @@ export function Settings({ onClose, onSave }: { onClose: () => void; onSave: ()
             </select>
           </Field>
           <label className="check">
-            <input type="checkbox" checked={settings.bleed} onChange={(e) => setSettings({ bleed: e.target.checked })} />
+            <input type="checkbox" checked={draft.bleed} onChange={(e) => set({ bleed: e.target.checked })} />
             Add bleed for full-page images (print)
           </label>
         </div>
         <div className="panel-foot">
-          <button
-            className="btn-primary"
-            onClick={() => {
-              onSave();
-              onClose();
-            }}
-          >
+          <button className="btn-primary" onClick={save}>
             Save
           </button>
         </div>