From 1bceeeb2f615ebc52066121a5f5234096bd0c06b Mon Sep 17 00:00:00 2001
From: Gabe Farrell <gabe@mnrva.dev>
Date: Mon, 24 Nov 2025 23:57:52 -0500
Subject: [PATCH] fix: add message to suggest dnd local file

---
 .../components/modals/ImageReplaceModal.tsx   | 179 ++++++++++--------
 1 file changed, 102 insertions(+), 77 deletions(-)

diff --git a/client/app/components/modals/ImageReplaceModal.tsx b/client/app/components/modals/ImageReplaceModal.tsx
index d76dd61..53954f5 100644
--- a/client/app/components/modals/ImageReplaceModal.tsx
+++ b/client/app/components/modals/ImageReplaceModal.tsx
@@ -5,86 +5,111 @@ import SearchResults from "../SearchResults";
 import { AsyncButton } from "../AsyncButton";
 
 interface Props {
-    type: string
-    id: number
-    musicbrainzId?: string
-    open: boolean 
-    setOpen: Function
+  type: string;
+  id: number;
+  musicbrainzId?: string;
+  open: boolean;
+  setOpen: Function;
 }
 
-export default function ImageReplaceModal({ musicbrainzId, type, id, open, setOpen }: Props) {
-    const [query, setQuery] = useState('');
-    const [loading, setLoading] = useState(false)
-    const [suggestedImgLoading, setSuggestedImgLoading] = useState(true)
+export default function ImageReplaceModal({
+  musicbrainzId,
+  type,
+  id,
+  open,
+  setOpen,
+}: Props) {
+  const [query, setQuery] = useState("");
+  const [loading, setLoading] = useState(false);
+  const [error, setError] = useState("");
+  const [suggestedImgLoading, setSuggestedImgLoading] = useState(true);
 
-    const doImageReplace = (url: string) => {
-        setLoading(true)
-        const formData = new FormData
-        formData.set(`${type.toLowerCase()}_id`, id.toString())
-        formData.set("image_url", url)
-        replaceImage(formData)
-        .then((r) => {
-            if (r.ok) {
-                window.location.reload()
-            } else {
-                console.log(r)
-                setLoading(false)
-            }
-        })
-        .catch((err) => console.log(err))
-    }
+  const doImageReplace = (url: string) => {
+    setLoading(true);
+    setError("");
+    const formData = new FormData();
+    formData.set(`${type.toLowerCase()}_id`, id.toString());
+    formData.set("image_url", url);
+    replaceImage(formData)
+      .then((r) => {
+        if (r.status >= 200 && r.status < 300) {
+          window.location.reload();
+        } else {
+          r.json().then((r) => setError(r.error));
+          setLoading(false);
+        }
+      })
+      .catch((err) => setError(err));
+  };
 
-    const closeModal = () => {
-        setOpen(false)
-        setQuery('')
-    }
+  const closeModal = () => {
+    setOpen(false);
+    setQuery("");
+    setError("");
+  };
 
-    return (
-        <Modal isOpen={open} onClose={closeModal}>
-            <h2>Replace Image</h2>
-            <div className="flex flex-col items-center">
-                <input
-                    type="text"
-                    autoFocus
-                    // i find my stupid a(n) logic to be a little silly so im leaving it in even if its not optimal
-                    placeholder={`Image URL`}
-                    className="w-full mx-auto fg bg rounded p-2"
-                    value={query}
-                    onChange={(e) => setQuery(e.target.value)}
+  return (
+    <Modal isOpen={open} onClose={closeModal}>
+      <h2>Replace Image</h2>
+      <div className="flex flex-col items-center">
+        <input
+          type="text"
+          autoFocus
+          // i find my stupid a(n) logic to be a little silly so im leaving it in even if its not optimal
+          placeholder={`Enter image URL, or drag-and-drop a local file`}
+          className="w-full mx-auto fg bg rounded p-2"
+          value={query}
+          onChange={(e) => setQuery(e.target.value)}
+        />
+        {query != "" ? (
+          <div className="flex gap-2 mt-4">
+            <AsyncButton
+              loading={loading}
+              onClick={() => doImageReplace(query)}
+            >
+              Submit
+            </AsyncButton>
+          </div>
+        ) : (
+          ""
+        )}
+        {type === "Album" && musicbrainzId ? (
+          <>
+            <h3 className="mt-5">Suggested Image (Click to Apply)</h3>
+            <button
+              className="mt-4"
+              disabled={loading}
+              onClick={() =>
+                doImageReplace(
+                  `https://coverartarchive.org/release/${musicbrainzId}/front`
+                )
+              }
+            >
+              <div className={`relative`}>
+                {suggestedImgLoading && (
+                  <div className="absolute inset-0 flex items-center justify-center">
+                    <div
+                      className="animate-spin rounded-full border-2 border-gray-300 border-t-transparent"
+                      style={{ width: 20, height: 20 }}
+                    />
+                  </div>
+                )}
+                <img
+                  src={`https://coverartarchive.org/release/${musicbrainzId}/front`}
+                  onLoad={() => setSuggestedImgLoading(false)}
+                  onError={() => setSuggestedImgLoading(false)}
+                  className={`block w-[130px] h-auto ${
+                    suggestedImgLoading ? "opacity-0" : "opacity-100"
+                  } transition-opacity duration-300`}
                 />
-                { query != "" ? 
-                <div className="flex gap-2 mt-4">
-                    <AsyncButton loading={loading} onClick={() => doImageReplace(query)}>Submit</AsyncButton>
-                </div> :
-                ''}
-                { type === "Album" && musicbrainzId ?
-                <>
-                <h3 className="mt-5">Suggested Image (Click to Apply)</h3>
-                <button 
-                    className="mt-4"
-                    disabled={loading}
-                    onClick={() => doImageReplace(`https://coverartarchive.org/release/${musicbrainzId}/front`)}
-                >
-                    <div className={`relative`}>
-                    {suggestedImgLoading && (
-                    <div className="absolute inset-0 flex items-center justify-center">
-                        <div
-                        className="animate-spin rounded-full border-2 border-gray-300 border-t-transparent"
-                        style={{ width: 20, height: 20 }}
-                        />
-                    </div>
-                    )}
-                    <img
-                    src={`https://coverartarchive.org/release/${musicbrainzId}/front`}
-                    onLoad={() => setSuggestedImgLoading(false)}
-                    onError={() => setSuggestedImgLoading(false)}
-                    className={`block w-[130px] h-auto ${suggestedImgLoading ? 'opacity-0' : 'opacity-100'} transition-opacity duration-300`} />
-                    </div>
-                </button>
-                </>
-                : ''
-                }
-            </div>
-        </Modal>
-    )
-}
\ No newline at end of file
+              </div>
+            </button>
+          </>
+        ) : (
+          ""
+        )}
+        <p className="error">{error}</p>
+      </div>
+    </Modal>
+  );
+}