From aa5333dcd9020b82c5919f9532f85a38e4be102a Mon Sep 17 00:00:00 2001
From: Reece <reece@stirlingpdf.com>
Date: Fri, 26 Sep 2025 03:23:59 +0100
Subject: [PATCH] Change to button based placement to avoid performance issue
 on canvas

---
 .../components/tools/sign/SignSettings.tsx    | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/frontend/src/components/tools/sign/SignSettings.tsx b/frontend/src/components/tools/sign/SignSettings.tsx
index 0f234c69e..775f05fbb 100644
--- a/frontend/src/components/tools/sign/SignSettings.tsx
+++ b/frontend/src/components/tools/sign/SignSettings.tsx
@@ -90,12 +90,6 @@ const SignSettings = ({ parameters, onParameterChange, disabled = false, onActiv
       setCanvasSignatureData(dataURL);
       onParameterChange('signatureData', dataURL);
 
-      // Auto-activate placement mode after drawing
-      setTimeout(() => {
-        if (onActivateSignaturePlacement) {
-          onActivateSignaturePlacement();
-        }
-      }, 100);
     }
   };
 
@@ -255,13 +249,6 @@ const SignSettings = ({ parameters, onParameterChange, disabled = false, onActiv
     }
 
     setIsModalOpen(false);
-
-    // Auto-activate placement mode after saving modal signature
-    setTimeout(() => {
-      if (onActivateSignaturePlacement) {
-        onActivateSignaturePlacement();
-      }
-    }, 100);
   };
 
   // Handle signature image upload
@@ -564,7 +551,20 @@ const SignSettings = ({ parameters, onParameterChange, disabled = false, onActiv
               onMouseUp={stopDrawing}
               onMouseLeave={stopDrawing}
             />
-            <Group justify="flex-end">
+            <Group justify="space-between">
+              <Button
+                variant="filled"
+                color="blue"
+                size="compact-sm"
+                onClick={() => {
+                  if (onActivateSignaturePlacement) {
+                    onActivateSignaturePlacement();
+                  }
+                }}
+                disabled={disabled || !canvasSignatureData}
+              >
+                Update and Place
+              </Button>
               <Button
                 variant="subtle"
                 color="red"