From 714346a0c1c42c31b631e50356945f093c49b387 Mon Sep 17 00:00:00 2001
From: QSchlegel <quirin.schlegel@icloud.com>
Date: Mon, 24 Nov 2025 08:42:21 +0100
Subject: [PATCH] Update ReviewRequiredSignersCard to prevent state updates
 during editing

- Modified useEffect to update temporary required signers only when not in editing mode, enhancing user experience.
- Improved value handling in ToggleGroup to ensure valid numeric input and prevent invalid states.
---
 .../create/ReviewRequiredSignersCard.tsx        | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/components/pages/homepage/wallets/new-wallet-flow/create/ReviewRequiredSignersCard.tsx b/src/components/pages/homepage/wallets/new-wallet-flow/create/ReviewRequiredSignersCard.tsx
index 39cc9188..cb0a83dd 100644
--- a/src/components/pages/homepage/wallets/new-wallet-flow/create/ReviewRequiredSignersCard.tsx
+++ b/src/components/pages/homepage/wallets/new-wallet-flow/create/ReviewRequiredSignersCard.tsx
@@ -36,10 +36,12 @@ const ReviewRequiredSignersCard: React.FC<ReviewRequiredSignersCardProps> = ({
   const [isEditing, setIsEditing] = useState(false);
   const [tempNumRequired, setTempNumRequired] = useState(numRequiredSigners);
   
-  // Update temp states when props change
+  // Update temp states when props change (but not when editing)
   useEffect(() => {
-    setTempNumRequired(numRequiredSigners);
-  }, [numRequiredSigners]);
+    if (!isEditing) {
+      setTempNumRequired(numRequiredSigners);
+    }
+  }, [numRequiredSigners, isEditing]);
 
   // Exit edit mode if script type changes from atLeast to something else
   useEffect(() => {
@@ -102,9 +104,14 @@ const ReviewRequiredSignersCard: React.FC<ReviewRequiredSignersCardProps> = ({
                       <div className="w-full overflow-x-auto pb-2">
                         <ToggleGroup
                           type="single"
-                          value={tempNumRequired.toString()}
+                          value={tempNumRequired > 0 ? tempNumRequired.toString() : undefined}
                           onValueChange={(v) => {
-                            if (v) setTempNumRequired(Number(v));
+                            if (v && v !== "" && !isNaN(Number(v))) {
+                              const newValue = Number(v);
+                              if (newValue > 0 && newValue <= signersCount) {
+                                setTempNumRequired(newValue);
+                              }
+                            }
                           }}
                           className="justify-start flex-nowrap"
                         >