fix: BROS-478: Undo for Vector (#8835)

Co-authored-by: nick-skriabin <[email protected]>

Author: nick-skriabin

web/libs/editor/src/components/KonvaVector/KonvaVector.tsx

@@ -3216,6 +3216,8 @@ export const KonvaVector = forwardRef<KonvaVectorRef, KonvaVectorProps>((props,
     disabled,
     transformMode,
     disableInternalPointAddition,
+    handleTransformStart,
+    handleTransformEnd,
     pointCreationManager,
   });
 

web/libs/editor/src/components/KonvaVector/eventHandlers/mouseHandlers.ts

@@ -212,6 +212,8 @@ export function createMouseDownHandler(props: EventHandlerProps, handledSelectio
                 controlIndex: 1,
               });
               props.isDragging.current = true;
+              // Fire transform start event when control point dragging begins
+              props.handleTransformStart?.();
               props.lastPos.current = {
                 x: e.evt.clientX,
                 y: e.evt.clientY,
@@ -233,6 +235,8 @@ export function createMouseDownHandler(props: EventHandlerProps, handledSelectio
                 controlIndex: 2,
               });
               props.isDragging.current = true;
+              // Fire transform start event when control point dragging begins
+              props.handleTransformStart?.();
               props.lastPos.current = {
                 x: e.evt.clientX,
                 y: e.evt.clientY,
@@ -448,6 +452,8 @@ export function createMouseMoveHandler(props: EventHandlerProps, handledSelectio
       // If we haven't started dragging yet, check if we should start
       if (!props.isDragging.current && (mouseDeltaX > dragThreshold || mouseDeltaY > dragThreshold)) {
         props.isDragging.current = true;
+        // Fire transform start event when point dragging begins
+        props.handleTransformStart?.();
       }
 
       // Only proceed with dragging if we're actually dragging
@@ -775,6 +781,13 @@ export function createMouseUpHandler(props: EventHandlerProps) {
       }
     }
 
+    // Fire transform end event if we were dragging (point or control point)
+    const wasDragging =
+      props.isDragging.current || props.draggedPointIndex !== null || props.draggedControlPoint !== null;
+    if (wasDragging) {
+      props.handleTransformEnd?.(e);
+    }
+
     // Reset dragging state
     props.isDragging.current = false;
     props.setDraggedPointIndex(null);

web/libs/editor/src/components/KonvaVector/eventHandlers/types.ts

@@ -107,6 +107,8 @@ export interface EventHandlerProps {
   disabled?: boolean;
   transformMode?: boolean;
   disableInternalPointAddition?: boolean;
+  handleTransformStart?: () => void;
+  handleTransformEnd?: (e?: KonvaEventObject<MouseEvent>) => void;
   pointCreationManager?: {
     isCreating: () => boolean;
     createRegularPointAt: (x: number, y: number, prevPointId?: string) => boolean;

web/libs/editor/src/tools/Vector.js

@@ -264,6 +264,8 @@ const _Tool = types
 
         const currentArea = area && isAlive(area) ? area : null;
 
+        self.annotation.history.freeze();
+
         // Only create new region if we don't have an existing one
         if (!currentArea) {
           self.currentArea = self.createRegion(self.createRegionOptions(), true);
@@ -318,6 +320,7 @@ const _Tool = types
         // skipping a frame to let KonvaVector render and update properly
         setTimeout(() => {
           self.currentArea?.commitPoint?.(rx, ry);
+          self.annotation.history.unfreeze();
           self.finishDrawing();
         });
       },