1 files changed, 16 insertions, 21 deletions

diff --git a/js/tools/modifier-tool-base.js b/js/tools/modifier-tool-base.js
index 7892d015..94b806fd 100755
--- a/js/tools/modifier-tool-base.js
+++ b/js/tools/modifier-tool-base.js
@@ -134,7 +134,7 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
                                 var hitRec = snapManager.snap(point.x, point.y, do3DSnap);
 
                 // TODO - Check that hitRec's element matches element that browser says we clicked on
-                var elt = this.application.ninja.stage.GetElement(event);
+                var elt = this.application.ninja.stage.GetSelectableElement(event);
                 if(elt !== hitRec.getElement())
                 {
                     hitRec = snapManager.findHitRecordForElement(elt);
@@ -171,7 +171,7 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
 //                        }
 //                        else
 //                        {
-                            this._dragPlane = snapManager.setupDragPlanes( hitRec );
+                            this._dragPlane = snapManager.setupDragPlanes( hitRec, true );
 //                        }
                     }
 
@@ -552,6 +552,9 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
 
                     // update the target
                     this._mouseUpHitRec = hitRec;
+                    var pt = hitRec.getScreenPoint();
+                    this.upPoint.x = pt[0];
+                    this.upPoint.y = pt[1];
                                 }
                         }
         }
@@ -579,12 +582,15 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
 
             this.downPoint.x = null;
             this.downPoint.y = null;
+            this.upPoint.x = null;
+            this.upPoint.y = null;
 //            this.isDrawing = false;
 
             if(this._canSnap)
             {
                 this.cleanupSnap();
             }
+            this._mode = 0;
         }
     },
     
@@ -764,6 +770,8 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
         captureSelectionDrawn: {
                 value: function(event){
                         this._targets = [];
+            this._origin = null;
+            this._delta = null;
 
                         var len = this.application.ninja.selectedElements.length;
                         if(len)
@@ -828,14 +836,14 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
                 this.doDraw(event);
             } else {
                 this._showFeedbackOnMouseMove(event);
-//                if(this._canSnap)
-//                {
-//                    this.doSnap(event);
-//                }
+                if(this._canSnap)
+                {
+                    this.doSnap(event);
+                }
             }
 
             this.DrawHandles(this._delta);
-            if(this._canSnap && this._isDrawing)
+            if(this._canSnap)
             {
                 snapManager.drawLastHit();
             }
@@ -866,7 +874,6 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
                     this._updateTargets(true);
                 }
 
-                this.endDraw(event);
                 this._hasDraw = false;
             }
             if(this._handleMode !== null)
@@ -874,6 +881,7 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
                 this._handleMode = null;
                 this._delta = null;
             }
+            this.endDraw(event);
             this.DrawHandles();
         }
     },
@@ -998,19 +1006,6 @@ exports.ModifierToolBase = Montage.create(DrawingTool, {
         }
     },
 
-        _updateDelta: {
-                value: function(delta, handleMode){
-                        if(this._clickedObject !== this.application.ninja.currentDocument.documentRoot)
-                        {
-                                this._delta += ~~(delta[handleMode]);
-                        }
-                        else
-                        {
-                                this._delta = ~~(delta[handleMode]);
-                        }
-                }
-        },
-
     modifyElements: {
         value: function(data, event) {
             // override in subclasses.