From 1b876bf8d345c8e5fb1bdd3ae0f51571b0294de8 Mon Sep 17 00:00:00 2001
From: Nicolas Mowen <nickmowen213@gmail.com>
Date: Tue, 13 Aug 2024 09:12:06 -0600
Subject: [PATCH] UI fixes (#13030)

* Fix difficulty overwriting export name

* Fix NaN for score selector
---
 web/src/components/card/ExportCard.tsx | 14 +++++++----
 web/src/pages/SubmitPlus.tsx           | 32 ++++++++++++++++----------
 2 files changed, 29 insertions(+), 17 deletions(-)

diff --git a/web/src/components/card/ExportCard.tsx b/web/src/components/card/ExportCard.tsx
index d39cbbeda..1ad98be0d 100644
--- a/web/src/components/card/ExportCard.tsx
+++ b/web/src/components/card/ExportCard.tsx
@@ -44,7 +44,7 @@ export default function ExportCard({
 
   const [editName, setEditName] = useState<{
     original: string;
-    update: string;
+    update?: string;
   }>();
 
   const submitRename = useCallback(() => {
@@ -52,7 +52,7 @@ export default function ExportCard({
       return;
     }
 
-    onRename(exportedRecording.id, editName.update);
+    onRename(exportedRecording.id, editName.update ?? "");
     setEditName(undefined);
   }, [editName, exportedRecording, onRename, setEditName]);
 
@@ -64,7 +64,7 @@ export default function ExportCard({
         modifiers.down &&
         !modifiers.repeat &&
         editName &&
-        editName.update.length > 0
+        (editName.update?.length ?? 0) > 0
       ) {
         submitRename();
       }
@@ -92,7 +92,11 @@ export default function ExportCard({
                 className="mt-3"
                 type="search"
                 placeholder={editName?.original}
-                value={editName?.update || editName?.original}
+                value={
+                  editName?.update == undefined
+                    ? editName?.original
+                    : editName?.update
+                }
                 onChange={(e) =>
                   setEditName({
                     original: editName.original ?? "",
@@ -159,7 +163,7 @@ export default function ExportCard({
                   onClick={() =>
                     setEditName({
                       original: exportedRecording.name,
-                      update: "",
+                      update: undefined,
                     })
                   }
                 >
diff --git a/web/src/pages/SubmitPlus.tsx b/web/src/pages/SubmitPlus.tsx
index 1fcc6ef2a..9d2b9ae5b 100644
--- a/web/src/pages/SubmitPlus.tsx
+++ b/web/src/pages/SubmitPlus.tsx
@@ -494,12 +494,16 @@ function PlusFilterGroup({
               className="w-12"
               inputMode="numeric"
               value={Math.round((currentScoreRange?.at(0) ?? 0.5) * 100)}
-              onChange={(e) =>
-                setCurrentScoreRange([
-                  parseInt(e.target.value) / 100.0,
-                  currentScoreRange?.at(1) ?? 1.0,
-                ])
-              }
+              onChange={(e) => {
+                const value = e.target.value;
+
+                if (value) {
+                  setCurrentScoreRange([
+                    parseInt(value) / 100.0,
+                    currentScoreRange?.at(1) ?? 1.0,
+                  ]);
+                }
+              }}
             />
             <DualThumbSlider
               className="w-full"
@@ -513,12 +517,16 @@ function PlusFilterGroup({
               className="w-12"
               inputMode="numeric"
               value={Math.round((currentScoreRange?.at(1) ?? 1.0) * 100)}
-              onChange={(e) =>
-                setCurrentScoreRange([
-                  currentScoreRange?.at(0) ?? 0.5,
-                  parseInt(e.target.value) / 100.0,
-                ])
-              }
+              onChange={(e) => {
+                const value = e.target.value;
+
+                if (value) {
+                  setCurrentScoreRange([
+                    currentScoreRange?.at(0) ?? 0.5,
+                    parseInt(value) / 100.0,
+                  ]);
+                }
+              }}
             />
           </div>
           <DropdownMenuSeparator />