From a71fc185309f8b1fdc1dc079b3f6185848189091 Mon Sep 17 00:00:00 2001
From: Yi Xiao <54782454+YIXIAO0@users.noreply.github.com>
Date: Fri, 23 Aug 2024 13:11:55 +0800
Subject: [PATCH] feat: set workflow zoom range for shortcut (#7563)

---
 .../workflow/hooks/use-shortcuts.ts           | 21 +++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/web/app/components/workflow/hooks/use-shortcuts.ts b/web/app/components/workflow/hooks/use-shortcuts.ts
index 9484f9c16e..666c3a45ba 100644
--- a/web/app/components/workflow/hooks/use-shortcuts.ts
+++ b/web/app/components/workflow/hooks/use-shortcuts.ts
@@ -37,12 +37,25 @@ export const useShortcuts = (): void => {
   const { handleLayout } = useWorkflowOrganize()
 
   const {
-    zoomIn,
-    zoomOut,
     zoomTo,
+    getZoom,
     fitView,
   } = useReactFlow()
 
+  // Zoom out to a minimum of 0.5 for shortcut
+  const constrainedZoomOut = () => {
+    const currentZoom = getZoom()
+    const newZoom = Math.max(currentZoom - 0.1, 0.5)
+    zoomTo(newZoom)
+  }
+
+  // Zoom in to a maximum of 1 for shortcut
+  const constrainedZoomIn = () => {
+    const currentZoom = getZoom()
+    const newZoom = Math.min(currentZoom + 0.1, 1)
+    zoomTo(newZoom)
+  }
+
   const shouldHandleShortcut = useCallback((e: KeyboardEvent) => {
     const { showFeaturesPanel } = workflowStore.getState()
     return !showFeaturesPanel && !isEventTargetInputArea(e.target as HTMLElement)
@@ -165,7 +178,7 @@ export const useShortcuts = (): void => {
   useKeyPress(`${getKeyboardKeyCodeBySystem('ctrl')}.dash`, (e) => {
     if (shouldHandleShortcut(e)) {
       e.preventDefault()
-      zoomOut()
+      constrainedZoomOut()
       handleSyncWorkflowDraft()
     }
   }, {
@@ -176,7 +189,7 @@ export const useShortcuts = (): void => {
   useKeyPress(`${getKeyboardKeyCodeBySystem('ctrl')}.equalsign`, (e) => {
     if (shouldHandleShortcut(e)) {
       e.preventDefault()
-      zoomIn()
+      constrainedZoomIn()
       handleSyncWorkflowDraft()
     }
   }, {