From 06970d710f7172ee5ab736ef082c7703c61bfd0c Mon Sep 17 00:00:00 2001
From: Pushkar Joshi
Date: Tue, 3 Apr 2012 11:13:07 -0700
Subject: track the canvas top left position instead of the center position to
 figure out when to translate the subpath per canvas translation...this one
 seems to not drift (no floating point issues uncovered so far)

---
 js/tools/PenTool.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

(limited to 'js/tools')

diff --git a/js/tools/PenTool.js b/js/tools/PenTool.js
index 0dbefd16..91b7606e 100755
--- a/js/tools/PenTool.js
+++ b/js/tools/PenTool.js
@@ -419,10 +419,6 @@ exports.PenTool = Montage.create(ShapeTool, {
                 bboxMax = this._selectedSubpath.getBBoxMax();
                 var bboxMid = [0.5 * (bboxMax[0] + bboxMin[0]), 0.5 * (bboxMax[1] + bboxMin[1]), 0.5 * (bboxMax[2] + bboxMin[2])];
 
-                this._selectedSubpath.setPlaneCenter(bboxMid);
-                this._selectedSubpath.setCanvasX(bboxMid[0]);
-                this._selectedSubpath.setCanvasY(bboxMid[1]);
-
                 //call render shape with the bbox width and height
                 this.RenderShape(bboxWidth, bboxHeight, bboxMid, this._penPlaneMat, this._penCanvas);
             }
@@ -437,6 +433,9 @@ exports.PenTool = Montage.create(ShapeTool, {
 
             var left = Math.round(midPt[0] - 0.5 * w);
             var top = Math.round(midPt[1] - 0.5 * h);
+            this._selectedSubpath.setPlaneCenter(midPt);
+            this._selectedSubpath.setCanvasLeft(left);
+            this._selectedSubpath.setCanvasTop(top);
 
             if (!canvas) {
                 var newCanvas = null;
-- 
cgit v1.2.3