labellife-design-tool 2.2.3 → 2.2.4

package/dist/cjs/canvas/workspace.js

@@ -2101,6 +2101,32 @@ var InputFieldsDialog = mobxReactLite.observer(function (_ref2) {
     }
   }, [fields.length]);
   var totalCount = initialTotalRef.current || fields.length;
+
+  // Back-from-summary: triggered by parent incrementing backTrigger prop
+  // Must be defined and its useEffect called BEFORE any early returns (Rules of Hooks)
+  var handleBackFromSummary = function handleBackFromSummary() {
+    var last = completedHistoryRef.current.pop();
+    if (!last) return;
+    var el = last.el,
+      previousValue = last.previousValue;
+    if (el) {
+      var isImage = el.custom && el.custom.inputType === 'image';
+      el.set(isImage ? {
+        src: previousValue || ''
+      } : {
+        text: previousValue || ''
+      });
+      isGoingBackRef.current = true;
+      store._unresolveInputField(el);
+    }
+    // displayIndex stays unchanged — it already points to the last step (N-1)
+  };
+  react.useEffect(function () {
+    if (backTrigger !== prevBackTriggerRef.current) {
+      prevBackTriggerRef.current = backTrigger;
+      handleBackFromSummary();
+    }
+  }, [backTrigger]);
   if (!enabled || fields.length === 0) return null;
 
   // Always show the first remaining field — the array shrinks as fields are resolved
@@ -2176,31 +2202,6 @@ var InputFieldsDialog = mobxReactLite.observer(function (_ref2) {
   };
   var canGoBack = displayIndex > 0;
 
-  // Back-from-summary: triggered by parent incrementing backTrigger prop
-  var handleBackFromSummary = function handleBackFromSummary() {
-    var last = completedHistoryRef.current.pop();
-    if (!last) return;
-    var el = last.el,
-      previousValue = last.previousValue;
-    if (el) {
-      var isImage = el.custom && el.custom.inputType === 'image';
-      el.set(isImage ? {
-        src: previousValue || ''
-      } : {
-        text: previousValue || ''
-      });
-      isGoingBackRef.current = true;
-      store._unresolveInputField(el);
-    }
-    // displayIndex stays unchanged — it already points to the last step (N-1)
-  };
-  react.useEffect(function () {
-    if (backTrigger !== prevBackTriggerRef.current) {
-      prevBackTriggerRef.current = backTrigger;
-      handleBackFromSummary();
-    }
-  }, [backTrigger]);
-
   // ── Resolve which dialog component to use ───────────────────────────
   // Priority: per-type custom dialog > global CustomDialog > built-in
   var TYPE_DIALOG_MAP = {