From 1d5852fbb56fe1ecccc2dca2138f9a886eb397a2 Mon Sep 17 00:00:00 2001
From: Ethan Garber <ethancgarber@gmail.com>
Date: Wed, 18 Jun 2025 21:26:40 -0400
Subject: [PATCH] fix ux concern to manage message state in dialog component

---
 web-app/src/containers/ThreadContent.tsx | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/web-app/src/containers/ThreadContent.tsx b/web-app/src/containers/ThreadContent.tsx
index 9ff3aedac..24eafbb67 100644
--- a/web-app/src/containers/ThreadContent.tsx
+++ b/web-app/src/containers/ThreadContent.tsx
@@ -79,6 +79,14 @@ const EditDialog = ({
   message: string
   setMessage: (message: string) => void
 }) => {
+  const [draft, setDraft] = useState(message)
+
+  const handleSave = () => {
+    if (draft !== message) {
+      setMessage(draft)
+    }
+  }
+
   return (
     <Dialog>
       <DialogTrigger>
@@ -97,10 +105,8 @@ const EditDialog = ({
         <DialogHeader>
           <DialogTitle>Edit Message</DialogTitle>
           <Textarea
-            value={message}
-            onChange={(e) => {
-              setMessage(e.target.value)
-            }}
+            value={draft}
+            onChange={(e) => setDraft(e.target.value)}
             className="mt-2 resize-none h-full w-full"
             onKeyDown={(e) => {
               // Prevent key from being captured by parent components
@@ -114,7 +120,12 @@ const EditDialog = ({
               </Button>
             </DialogClose>
             <DialogClose asChild>
-              <Button disabled={!message}>Save</Button>
+              <Button
+                disabled={draft === message || !draft}
+                onClick={handleSave}
+              >
+                Save
+              </Button>
             </DialogClose>
           </DialogFooter>
         </DialogHeader>