From 4d8622eed6c3ab21ba2038e84771d0538266ff19 Mon Sep 17 00:00:00 2001
From: Stewart Allen <stewart.allen@gmail.com>
Date: Sun, 15 Jun 2025 18:49:45 -0400
Subject: [PATCH] handle sketch and sketch item move undo/redo

---
 src2/main/mesh.js    |  1 -
 src2/mesh/api.js     |  2 ++
 src2/mesh/history.js | 72 ++++++++++++++++++++++++++------------------
 src2/mesh/sketch.js  | 20 ++++++++++--
 4 files changed, 62 insertions(+), 33 deletions(-)

diff --git a/src2/main/mesh.js b/src2/main/mesh.js
index 0c6d349e..ed44c639 100644
--- a/src2/main/mesh.js
+++ b/src2/main/mesh.js
@@ -475,7 +475,6 @@ function space_init(data) {
     space.mouse.onDrag((delta, offset, up = false) => {
         let { mode, modes } = api;
         if (delta) {
-            // if (delta.event?.shiftKey && selection.count()) {
             if (selection.count()) {
                 selection.drag({ delta, offset });
             }
diff --git a/src2/mesh/api.js b/src2/mesh/api.js
index c6ba8b54..ea8355a7 100644
--- a/src2/mesh/api.js
+++ b/src2/mesh/api.js
@@ -217,6 +217,8 @@ const selection = {
             broker.publish('drag_start', selected);
         } else if (opt.end) {
             broker.publish('drag_end', selected);
+        } else {
+            broker.publish('drag', opt);
         }
         selected.forEach(s => s.drag(opt));
         return selection;
diff --git a/src2/mesh/history.js b/src2/mesh/history.js
index 209e7f8d..770201b3 100644
--- a/src2/mesh/history.js
+++ b/src2/mesh/history.js
@@ -1,10 +1,13 @@
 /** Copyright Stewart Allen <sa@grid.space> -- All Rights Reserved */
 
 import { broker } from "../moto/broker.js";
+import { target } from "./sketch.js";
 
 class History {
     #pos = -1;
     #stack = [];
+    #offset; // store final drag move
+    #target; // store sketch drag target
     #tracking = false;
 
     constructor() {
@@ -14,26 +17,35 @@ class History {
                 this.#tracking = true
             },
 
-            drag_start(selected) {
-                this.push('position', selected.map(model => ({
-                    model,
-                    pos: model.position().toArray()
-                })));
+            drag_start(selected) {},
+
+            drag(opt) {
+                this.#target = target();
+                this.#offset = opt.offset;
             },
 
             drag_end(selected) {
-                this.update(selected.map(model => ({
-                    model,
-                    pos: model.position().toArray()
-                })));
+                // console.log({
+                //     drag_end: selected,
+                //     offset: this.#offset,
+                //     target: this.#target
+                // });
+                const { x, y, z } = this.#offset;
+                broker.publish('move', {
+                    target: this.#target,
+                    set: selected,
+                    dx: x,
+                    dy: y,
+                    dz: z
+                });
             },
 
             move(data) {
-                const { set, dx, dy, dz } = data;
+                const { target, set, dx, dy, dz } = data;
                 this.push('move', {
-                    set, dx: -dx, dy: -dy, dz: -dz
+                    set, dx: -dx, dy: -dy, dz: -dz, target
                 }, {
-                    set, dx, dy, dz
+                    set, dx, dy, dz, target
                 });
             },
 
@@ -67,32 +79,32 @@ class History {
         });
     }
 
-    update(redo, undo) {
-        let peek = this.#stack.peek();
-        if (peek) {
-            Object.assign(peek, {
-                redo: redo || peek.redo,
-                undo: undo || peek.undo
-            });
-        }
-    }
+    // update(redo, undo) {
+    //     let peek = this.#stack.peek();
+    //     if (peek) {
+    //         Object.assign(peek, {
+    //             redo: redo || peek.redo,
+    //             undo: undo || peek.undo
+    //         });
+    //     }
+    // }
 
     do(type, data) {
-        console.log({ type, data });
+        // console.log({ type, data });
         switch (type) {
             case 'move': {
-                let { set, dx, dy, dz } = data;
+                let { set, dx, dy, dz, target } = data;
                 for (let grp of set) {
-                    grp.move(dx, dy, dz);
-                }
-                break;
-            }
-            case 'position': {
-                for (let { model, pos } of data) {
-                    model.position(...pos);
+                    grp.move(dx, dy, dz, target);
                 }
                 break;
             }
+            // case 'position': {
+            //     for (let { model, pos } of data) {
+            //         model.position(...pos);
+            //     }
+            //     break;
+            // }
             case 'rotate': {
                 let { set, dx, dy, dz } = data;
                 for (let grp of set) {
diff --git a/src2/mesh/sketch.js b/src2/mesh/sketch.js
index 984fb55a..42fdd408 100644
--- a/src2/mesh/sketch.js
+++ b/src2/mesh/sketch.js
@@ -395,8 +395,20 @@ class MeshSketch extends meshObject {
         return bool;
     }
 
-    move(x, y, z = 0) {
+    position() {
         const { target } = drag;
+        console.log({ sk_pos: [ ...arguments ], target });
+        const [ nx, ny, nz ] = [ ...arguments ];
+        if (nx || ny || nz) {
+            this.center = { x: nx, y: ny, z: nz };
+            this.render();
+        } else {
+            const { x, y, z } = this.center;
+            return new Vector3(x, y, z);
+        }
+    }
+
+    move(x, y, z = 0, target = drag.target) {
         const { center, scale, plane, handles } = this;
         const handle = handles.indexOf(target);
         if (target === plane) {
@@ -636,4 +648,8 @@ class SketchItem {
     }
 }
 
-export { MeshSketch as sketch, SketchItem };
\ No newline at end of file
+function target() {
+    return drag.target;
+}
+
+export { MeshSketch as sketch, SketchItem, target };
\ No newline at end of file