2 files changed, 103 insertions, 128 deletions

diff --git a/js/lib/geom/anchor-point.js b/js/lib/geom/anchor-point.js
index 0e9f65ea..d254681c 100755
--- a/js/lib/geom/anchor-point.js
+++ b/js/lib/geom/anchor-point.js
@@ -221,6 +221,10 @@ GLAnchorPoint.prototype.getNext = function() {
     return [this._nextX, this._nextY, this._nextZ];
 };
 
+GLAnchorPoint.prototype.getAllPos = function() {
+    return [[this._prevX, this._prevY, this._prevZ],[this._x, this._y, this._z],[this._nextX, this._nextY, this._nextZ]];
+};
+
 //return the square of distance from passed in point to the anchor position
 GLAnchorPoint.prototype.getDistanceSq = function (x, y, z) {
     return (this._x - x) * (this._x - x) + (this._y - y) * (this._y - y) + (this._z - z) * (this._z - z);
diff --git a/js/tools/PenTool.js b/js/tools/PenTool.js
index ec63771a..127beacd 100755
--- a/js/tools/PenTool.js
+++ b/js/tools/PenTool.js
@@ -287,14 +287,7 @@ exports.PenTool = Montage.create(ShapeTool, {
                 // Compute the mouse position in local (selected subpath canvas) space
                 var globalPos = hitRec.getScreenPoint();
                 var localMousePos = ViewUtils.globalToLocal(globalPos, this._selectedSubpathCanvas);
-                /*
-                CHECK IF THIS IS CORRECT
-                ViewUtils.pushViewportObj
-                var temp ViewUtils.screenToView(localMousePos[0], localMousePos[1], 0) should return a point in view space of canvas
-                ViewUtils.popViewportObj
-                MathUtils.transformPoint(temp, this._selectedSubpathPlaneMat)
-                */
-
+                
                 //now perform the hit testing
                 var prevSelectedAnchorIndex = this._selectedSubpath.getSelectedAnchorIndex();
                 var selAnchorAndParamAndCode = this._selectedSubpath.pickPath(localMousePos[0], localMousePos[1], localMousePos[2], this._PICK_POINT_RADIUS, false);
@@ -417,29 +410,29 @@ exports.PenTool = Montage.create(ShapeTool, {
 
             var hitRec = this.getHitRecord(event.pageX, event.pageY);
 
-            if (this._isDrawing) {
-                if (0){
-                //if (currMousePos && this._selectedSubpath && (this._selectedSubpath.getSelectedAnchorIndex() >= 0 && this._selectedSubpath.getSelectedAnchorIndex() < this._selectedSubpath.getNumAnchors())) {
-                    // BEGIN NEW LOCAL COORD BLOCK
-                    //build the mouse position in local coordinates
-                    var drawingCanvas = this._selectedSubpath.getCanvas();
-                    if (!drawingCanvas){
-                        drawingCanvas = ViewUtils.getStageElement();
-                    }
-                    posInfo = this._getMouseEventPosition (event.pageX, event.pageY, false, false);
-                    var globalMousePos = posInfo[0];
-                    var localMousePos = ViewUtils.globalToLocal(globalMousePos, drawingCanvas);
+            var drawingCanvas=null, globalMousePos=null, localMousePos=null;
+            if (this._selectedSubpath ){
+                drawingCanvas = this._selectedSubpath.getCanvas();
+                if (!drawingCanvas){
+                    drawingCanvas = ViewUtils.getStageElement();
+                }
+                globalMousePos = hitRec.getScreenPoint();
+                localMousePos = ViewUtils.globalToLocal(globalMousePos, drawingCanvas);
+            }
 
+            if (this._isDrawing) {
+                //if there is a selected subpath with a selected anchor point
+                if (this._selectedSubpath.getSelectedAnchorIndex() >= 0 && this._selectedSubpath.getSelectedAnchorIndex() < this._selectedSubpath.getNumAnchors()) {
                     //compute the translation from the selected anchor
-                    var selAnchorLocalPos = this._selectedSubpath.getAnchorLocalCoord(this._selectedSubpath.getSelectedAnchorIndex());
-                    var localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorLocalPos[1]);
                     var selAnchor = this._selectedSubpath.getAnchor(this._selectedSubpath.getSelectedAnchorIndex());
+                    var selAnchorPos = selAnchor.getAllPos();
+                    var localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorPos[1]);
                     
                     if (this._editMode & this.EDIT_ANCHOR) {
                         selAnchor.translateAll(localTranslation[0], localTranslation[1], localTranslation[2]);
                     }
                     else if (this._editMode & this.EDIT_PREV) {
-                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorLocalPos[0]);
+                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorPos[0]);
                         selAnchor.translatePrev(localTranslation[0], localTranslation[1], localTranslation[2]);
 
                         //move the next point if Alt key is down to ensure relative angle between prev and next
@@ -448,7 +441,7 @@ exports.PenTool = Montage.create(ShapeTool, {
                         }
                     }
                     else if (this._editMode & this.EDIT_NEXT) {
-                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorLocalPos[2]);
+                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorPos[2]);
                         selAnchor.translateNext(localTranslation[0], localTranslation[1], localTranslation[2]);
 
                         //move the prev point if Alt key is down to ensure relative angle between prev and next
@@ -457,12 +450,11 @@ exports.PenTool = Montage.create(ShapeTool, {
                         }
                     }
                     else if (this._editMode & this.EDIT_PREV_NEXT) {
-                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorLocalPos[2]);
+                        localTranslation = VecUtils.vecSubtract(3, localMousePos, selAnchorPos[2]);
                         selAnchor.translateNext(localTranslation[0], localTranslation[1], localTranslation[2]);
                         selAnchor.setPrevFromNext();
                     }
 
-
                     //snapping...check if the new location of the anchor point is close to another anchor point
                     var selX = selAnchor.getPosX();
                     var selY = selAnchor.getPosY();
@@ -482,59 +474,37 @@ exports.PenTool = Montage.create(ShapeTool, {
                         }
                     }
 
-                    // END NEW LOCAL COORD BLOCK
-
                     //make the subpath dirty so it will get re-drawn
                     this._selectedSubpath.makeDirty();
                     this.DrawSubpathOnStage(this._selectedSubpath);
                 }
             } else { //if mouse is not down:
-                //this.doSnap(event);
-                //this.DrawHandles();
-
-                if (this._selectedSubpath ){
-
-                    //convert the mouse pos. to local space of the canvas
-                    //var widthAdjustment = -snapManager.getStageWidth()*0.5;
-                    //var heightAdjustment = -snapManager.getStageHeight()*0.5;
-
-
-                    var drawingCanvas = this._selectedSubpath.getCanvas();
-                    if (!drawingCanvas){
-                        drawingCanvas = ViewUtils.getStageElement();
+                var selAnchorAndParamAndCode = this._selectedSubpath.pickPath(localMousePos[0], localMousePos[1], localMousePos[2], this._PICK_POINT_RADIUS, true);
+                if (selAnchorAndParamAndCode[0] >=0){ //something on the path was hit
+                    if ((selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_ANCHOR)
+                        || (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_PREV)
+                        || (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_NEXT))
+                    { //the anchor was hit
+                        this._hoveredAnchorIndex = selAnchorAndParamAndCode[0];
+                        var lastAnchorIndex = this._selectedSubpath.getNumAnchors()-1;
+                        var cursor = "url('images/cursors/penCursors/Pen_anchorSelect.png') 5 1, default";
+                        if (this._selectedSubpath.getIsClosed()===false){
+                            if (this._entryEditMode === this.ENTRY_SELECT_PATH && !this._isPickedEndPointInSelectPathMode && (this._hoveredAnchorIndex===0 || this._hoveredAnchorIndex===lastAnchorIndex)){
+                                //if we're in SELECT_PATH mode, have not yet clicked on the end anchors,  AND we hovered over one of the end anchors
+                                cursor = "url('images/cursors/penCursors/Pen_append.png') 5 1, default";
+                            } else if ( this._selectedSubpath.getSelectedAnchorIndex()===lastAnchorIndex && this._hoveredAnchorIndex===0)  {
+                                //if we've selected the last anchor and hover over the first anchor
+                                cursor = "url('images/cursors/penCursors/Pen_closePath.png') 5 1, default";
+                            }
+                        } //if path is not closed
+                        this.application.ninja.stage.drawingCanvas.style.cursor = cursor;
+                    } else if (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_PATH) {
+                        //change the cursor
+                        var cursor = "url('images/cursors/penCursors/Pen_plus.png') 5 1, default";
+                        this.application.ninja.stage.drawingCanvas.style.cursor = cursor;
                     }
-                    var globalMousePos = hitRec.getScreenPoint();
-                    var localMousePos = ViewUtils.globalToLocal(globalMousePos, drawingCanvas);
-
-                    //var selAnchorRetCode = this._selectedSubpath.pickAnchor(currMousePos[0], currMousePos[1], currMousePos[2], this._PICK_POINT_RADIUS, false);
-                    //var selAnchorRetCode = this._selectedSubpath.pickAnchor(localMousePos[0], localMousePos[1], localMousePos[2], this._PICK_POINT_RADIUS);
-                    var selAnchorAndParamAndCode = this._selectedSubpath.pickPath(localMousePos[0], localMousePos[1], localMousePos[2], this._PICK_POINT_RADIUS, true);
-                    if (selAnchorAndParamAndCode[0] >=0){ //something on the path was hit
-                        if ((selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_ANCHOR)
-                            || (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_PREV)
-                            || (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_NEXT))
-                        { //the anchor was hit
-                            this._hoveredAnchorIndex = selAnchorAndParamAndCode[0];
-                            var lastAnchorIndex = this._selectedSubpath.getNumAnchors()-1;
-                            var cursor = "url('images/cursors/penCursors/Pen_anchorSelect.png') 5 1, default";
-                            if (this._selectedSubpath.getIsClosed()===false){
-                                if (this._entryEditMode === this.ENTRY_SELECT_PATH && !this._isPickedEndPointInSelectPathMode && (this._hoveredAnchorIndex===0 || this._hoveredAnchorIndex===lastAnchorIndex)){
-                                    //if we're in SELECT_PATH mode, have not yet clicked on the end anchors,  AND we hovered over one of the end anchors
-                                    cursor = "url('images/cursors/penCursors/Pen_append.png') 5 1, default";
-                                } else if ( this._selectedSubpath.getSelectedAnchorIndex()===lastAnchorIndex && this._hoveredAnchorIndex===0)  {
-                                    //if we've selected the last anchor and hover over the first anchor
-                                    cursor = "url('images/cursors/penCursors/Pen_closePath.png') 5 1, default";
-                                }
-                            } //if path is not closed
-                            this.application.ninja.stage.drawingCanvas.style.cursor = cursor;
-                        } else if (selAnchorAndParamAndCode[2] & this._selectedSubpath.SEL_PATH) {
-                            //change the cursor
-                            var cursor = "url('images/cursors/penCursors/Pen_plus.png') 5 1, default";
-                            this.application.ninja.stage.drawingCanvas.style.cursor = cursor;
-                        }
-                    } //something on the path was hit
-                }
-            } //else of if (this._isDrawing) {
+                } //something on the path was hit
+            } //mouse is not down
 
             //this.drawLastSnap();        // Required cleanup for both Draw/Feedbacks
             if (this._selectedSubpath){
@@ -781,24 +751,22 @@ exports.PenTool = Montage.create(ShapeTool, {
                 var bboxMid = VecUtils.vecInterpolate(3, bboxMin, bboxMax, 0.5);
 
                 //sandwich the planeMat between with the translation to the previous center of the canvas in local space and its inverse
-                var centerDisp = VecUtils.vecSubtract(3, bboxMid, this._selectedSubpathLocalCenter);
-                var tMat = Matrix.Translation([centerDisp[0], centerDisp[1],centerDisp[2]]);
-                var tInvMat = Matrix.Translation([-centerDisp[0], -centerDisp[1], -centerDisp[2]]);
-                var newMat = Matrix.I(4);
-                            glmat4.multiply( tInvMat, this._selectedSubpathPlaneMat, newMat);
-                glmat4.multiply( newMat, tMat, newMat);
-                this._selectedSubpathPlaneMat = newMat;
-                ViewUtils.setMatrixForElement(this._selectedSubpathCanvas, newMat, true);
+                if (this._selectedSubpathLocalCenter) {
+                    var centerDisp = VecUtils.vecSubtract(3, bboxMid, this._selectedSubpathLocalCenter);
+                    var tMat = Matrix.Translation([centerDisp[0], centerDisp[1],centerDisp[2]]);
+                    var tInvMat = Matrix.Translation([-centerDisp[0], -centerDisp[1], -centerDisp[2]]);
+                    var newMat = Matrix.I(4);
+                    glmat4.multiply( tInvMat, this._selectedSubpathPlaneMat, newMat);
+                    glmat4.multiply( newMat, tMat, newMat);
+                    this._selectedSubpathPlaneMat = newMat;