Cleans up the new-doc forms.

src/components/documents/location/NewLocationForm.tsx

@@ -1,6 +1,9 @@
 import { useState } from "react";
 import type { CampaignId, Location } from "@/lib/types";
 import { pb } from "@/lib/pocketbase";
+import { BaseForm } from "@/components/form/BaseForm";
+import { MultiLineInput } from "@/components/form/MultiLineInput";
+import { SingleLineInput } from "@/components/form/SingleLineInput";
 
 /**
  * Renders a form to add a new location. Calls onCreate with the new location document.
@@ -42,42 +45,29 @@ export const NewLocationForm = ({
   }
 
   return (
-    <form
+    <BaseForm
-      className="flex items-left flex-col gap-2 mt-4"
+      title="Create new Location"
       onSubmit={handleSubmit}
-    >
+      isLoading={adding || !name.trim()}
-      <h3>Create new location</h3>
+      error={error}
-      <div className="flex gap-5 w-full items-center">
+      content={
-        <label>Name</label>
+        <>
-        <input
+          <SingleLineInput
-          type="text"
+            label="Name"
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
+            value={name}
-          placeholder="Name"
+            onChange={setName}
-          value={name}
+            disabled={adding}
-          onChange={(e) => setName(e.target.value)}
+            placeholder="Enter location name"
-          disabled={adding}
+          />
-          aria-label="Name"
+          <MultiLineInput
-        />
+            label="Description"
-      </div>
+            value={description}
-      <div className="flex gap-5 w-full items-center">
+            placeholder="Enter location description"
-        <label>Description</label>
+            onChange={setDescription}
-        <textarea
+            disabled={adding}
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
+          />
-          placeholder="Description"
+        </>
-          value={description}
+      }
-          onChange={(e) => setDescription(e.target.value)}
+    />
-          disabled={adding}
-          aria-label="Description"
-        />
-      </div>
-      {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
-      <button
-        type="submit"
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
-        disabled={adding || !name.trim()}
-      >
-        {adding ? "Adding..." : "Create"}
-      </button>
-    </form>
   );
 };

src/components/documents/monsters/NewMonsterForm.tsx

@@ -1,6 +1,8 @@
 import { useState } from "react";
 import type { CampaignId, Monster } from "@/lib/types";
 import { pb } from "@/lib/pocketbase";
+import { BaseForm } from "@/components/form/BaseForm";
+import { SingleLineInput } from "@/components/form/SingleLineInput";
 
 /**
  * Renders a form to add a new monster. Calls onCreate with the new monster document.
@@ -13,7 +15,6 @@ export const NewMonsterForm = ({
   onCreate: (monster: Monster) => Promise<void>;
 }) => {
   const [name, setName] = useState("");
-  const [description, setDescription] = useState("");
   const [adding, setAdding] = useState(false);
   const [error, setError] = useState<string | null>(null);
 
@@ -28,11 +29,9 @@ export const NewMonsterForm = ({
         type: "monster",
         data: {
           name,
-          description,
         },
       });
       setName("");
-      setDescription("");
       await onCreate(monsterDoc);
     } catch (e: any) {
       setError(e?.message || "Failed to add monster.");
@@ -42,31 +41,18 @@ export const NewMonsterForm = ({
   }
 
   return (
-    <form
+    <BaseForm
-      className="flex items-left flex-col gap-2 mt-4"
+      title="Create new monster"
+      isLoading={adding || !name.trim()}
       onSubmit={handleSubmit}
-    >
+      error={error}
-      <h3>Create new monster</h3>
+      content={
-      <div className="flex gap-5 w-full align-center">
+        <SingleLineInput
-        <label>Name</label>
-        <input
-          type="text"
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
-          placeholder="Name"
           value={name}
-          onChange={(e) => setName(e.target.value)}
+          onChange={setName}
-          disabled={adding}
+          placeholder="Monster description"
-          aria-label="Name"
         />
-      </div>
+      }
-      {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
+    />
-      <button
-        type="submit"
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
-        disabled={adding || !name.trim()}
-      >
-        {adding ? "Adding..." : "Create"}
-      </button>
-    </form>
   );
 };

src/components/documents/npc/NewNpcForm.tsx

@@ -1,6 +1,9 @@
 import { useState } from "react";
 import type { CampaignId, Npc } from "@/lib/types";
 import { pb } from "@/lib/pocketbase";
+import { BaseForm } from "@/components/form/BaseForm";
+import { SingleLineInput } from "@/components/form/SingleLineInput";
+import { MultiLineInput } from "@/components/form/MultiLineInput";
 
 /**
  * Renders a form to add a new npc. Calls onCreate with the new npc document.
@@ -42,42 +45,29 @@ export const NewNpcForm = ({
   }
 
   return (
-    <form
+    <BaseForm
-      className="flex items-left flex-col gap-2 mt-4"
+      title="Create new NPC"
       onSubmit={handleSubmit}
-    >
+      isLoading={adding}
-      <h3>Create new npc</h3>
+      error={error}
-      <div className="flex gap-5 w-full items-center">
+      content={
-        <label>Name</label>
+        <>
-        <input
+          <SingleLineInput
-          type="text"
+            label="Name"
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
+            value={name}
-          placeholder="Name"
+            onChange={setName}
-          value={name}
+            disabled={adding}
-          onChange={(e) => setName(e.target.value)}
+            placeholder="Enter NPC name"
-          disabled={adding}
+          />
-          aria-label="Name"
+          <MultiLineInput
-        />
+            label="Description"
-      </div>
+            value={description}
-      <div className="flex gap-5 w-full items-center">
+            placeholder="Enter NPC description"
-        <label>Description</label>
+            onChange={setDescription}
-        <textarea
+            disabled={adding}
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
+          />
-          placeholder="Description"
+        </>
-          value={description}
+      }
-          onChange={(e) => setDescription(e.target.value)}
+    />
-          disabled={adding}
-          aria-label="Description"
-        />
-      </div>
-      {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
-      <button
-        type="submit"
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
-        disabled={adding || !name.trim()}
-      >
-        {adding ? "Adding..." : "Create"}
-      </button>
-    </form>
   );
 };

src/components/documents/secret/NewSecretForm.tsx

@@ -3,6 +3,8 @@
 import { useState } from "react";
 import type { CampaignId, Secret } from "@/lib/types";
 import { pb } from "@/lib/pocketbase";
+import { BaseForm } from "@/components/form/BaseForm";
+import { SingleLineInput } from "@/components/form/SingleLineInput";
 
 /**
  * Renders a form to add a new secret. Calls onCreate with the new secret document.
@@ -42,24 +44,18 @@ export const NewSecretForm = ({
   }
 
   return (
-    <form className="flex items-center gap-2 mt-4" onSubmit={handleSubmit}>
+    <BaseForm
-      <input
+      title="Create new treasure"
-        type="text"
+      isLoading={adding || !newSecret.trim()}
-        className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
+      onSubmit={handleSubmit}
-        placeholder="Add a new secret..."
+      error={error}
-        value={newSecret}
+      content={
-        onChange={(e) => setNewSecret(e.target.value)}
+        <SingleLineInput
-        disabled={adding}
+          value={newSecret}
-        aria-label="Add new secret"
+          onChange={setNewSecret}
-      />
+          placeholder="Treasure description"
-      {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
+        />
-      <button
+      }
-        type="submit"
+    />
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
-        disabled={adding || !newSecret.trim()}
-      >
-        {adding ? "Adding..." : "Add Secret"}
-      </button>
-    </form>
   );
 };

src/components/documents/treasure/NewTreasureForm.tsx

@@ -3,6 +3,8 @@
 import { useState } from "react";
 import type { CampaignId, Treasure } from "@/lib/types";
 import { pb } from "@/lib/pocketbase";
+import { BaseForm } from "@/components/form/BaseForm";
+import { SingleLineInput } from "@/components/form/SingleLineInput";
 
 /**
  * Renders a form to add a new treasure. Calls onCreate with the new treasure document.
@@ -42,29 +44,18 @@ export const NewTreasureForm = ({
   }
 
   return (
-    <form
+    <BaseForm
-      className="flex flex-col items-center gap-2 mt-4 w-full"
+      title="Create new treasure"
+      isLoading={adding || !newTreasure.trim()}
       onSubmit={handleSubmit}
-    >
+      error={error}
-      <div>
+      content={
-        <input
+        <SingleLineInput
-          type="text"
-          className="flex-1 px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
-          placeholder="Add a new treasure..."
           value={newTreasure}
-          onChange={(e) => setNewTreasure(e.target.value)}
+          onChange={setNewTreasure}
-          disabled={adding}
+          placeholder="Treasure description"
-          aria-label="Add new treasure"
         />
-      </div>
+      }
-      {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
+    />
-      <button
-        type="submit"
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
-        disabled={adding || !newTreasure.trim()}
-      >
-        {adding ? "Adding..." : "Add Treasure"}
-      </button>
-    </form>
   );
 };

src/components/form/BaseForm.tsx

@@ -17,12 +17,12 @@ export const BaseForm = ({
 }: Props) => {
   return (
     <form className="flex flex-col items-left gap-2" onSubmit={onSubmit}>
-      <h3 className="text-lg font-semibold text-slate-100 mb-4">{title}</h3>
+      <h3 className="text-lg font-semibold text-slate-100">{title}</h3>
-      <div>{content}</div>
+      <div className="flex flex-col gap-2 w-full items-stretch">{content}</div>
       {error && <div className="text-red-400 mt-2 text-sm">{error}</div>}
       <button
         type="submit"
-        className="px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
+        className="mt-2 px-4 py-2 rounded bg-emerald-600 hover:bg-emerald-700 text-white font-semibold transition-colors disabled:opacity-60"
         disabled={isLoading}
       >
         {buttonText ? buttonText : "Submit"}

src/components/form/MultiLineInput.tsx

@@ -1,6 +1,7 @@
 export type Props = {
   value: string;
   onChange: (value: string) => void;
+  label?: string;
   className?: string;
 } & Omit<
   React.TextareaHTMLAttributes<HTMLTextAreaElement>,
@@ -11,12 +12,17 @@ export const MultiLineInput = ({
   value,
   onChange,
   className = "",
+  label,
   ...props
 }: Props) => (
-  <textarea
+  <>
-    value={value}
+    {label && <label>{label}</label>}
-    onChange={(e) => onChange(e.target.value)}
+    <textarea
-    className={`w-full min-h-[10em] field-sizing-content p-2 rounded border bg-slate-800 text-slate-100 border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500 transition-colors ${className}`}
+      value={value}
-    {...props}
+      onChange={(e) => onChange(e.target.value)}
-  />
+      className={`w-full min-h-[10em] field-sizing-content p-2 rounded border bg-slate-800 text-slate-100 border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500 transition-colors ${className}`}
+      aria-label={label}
+      {...props}
+    />
+  </>
 );

src/components/form/SingleLineInput.tsx

@@ -0,0 +1,29 @@
+export type Props = {
+  value: string;
+  onChange: (value: string) => void;
+  label?: string;
+  className?: string;
+} & Omit<
+  React.InputHTMLAttributes<HTMLInputElement>,
+  "value" | "onChange" | "className"
+>;
+
+export const SingleLineInput = ({
+  value,
+  onChange,
+  className = "",
+  label,
+  ...props
+}: Props) => (
+  <>
+    {label && <label>{label}</label>}
+    <input
+      type="text"
+      value={value}
+      onChange={(e) => onChange(e.target.value)}
+      className={`w-full p-2 rounded border bg-slate-800 text-slate-100 border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500 transition-colors ${className}`}
+      aria-label={label}
+      {...props}
+    />
+  </>
+);