1 files changed, 74 insertions, 106 deletions

diff --git a/js/tools/Translate3DToolBase.js b/js/tools/Translate3DToolBase.js
index 4dca6313..04359567 100755
--- a/js/tools/Translate3DToolBase.js
+++ b/js/tools/Translate3DToolBase.js
@@ -21,14 +21,15 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
         _clickedOnStage: { value: false },
 
         HandleDoubleClick : {
-           value : function()
-           {
+           value : function() {
            }
         },
 
     modifyElements : {
                 value : function(data, event)
         {
+            //console.log( "modifyElements, data: " + data.pt0 + " => " + data.pt1 );
+
             // form the translation vector and post translate the matrix by it.
             var delta = vecUtils.vecSubtract( 3, data.pt1, data.pt0 );
             if(this._handleMode !== null)
@@ -66,16 +67,13 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
                     delta[0] = 0;
                     delta[1] = 0;
                 }
-                else
-                {
-                    delta[2] = 0;
-                }
                 this._delta = delta.slice(0);
             }
 
             var transMat = Matrix.Translation( delta );
 
-            if(this._inLocalMode && (this._targets.length === 1) )
+            //console.log( "Translate: " + delta );
+            if(this._inLocalMode && (this.application.ninja.selectedElements.length === 1) )
             {
                 this._translateLocally(transMat);
             }
@@ -87,53 +85,45 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
         },
 
 
-        Reset : {
-                value : function()
-                {
-            var item,
-                elt,
-                mat,
-                dist,
-                newStyles = [],
-                previousStyles = [],
-                len = this._targets.length;
-            for(var i = 0; i < len; i++)
-            {
-                // Reset to the identity matrix but retain the rotation values
-                item = this._targets[i];
-                mat = item.mat.slice(0);
+        Reset: {
+                value: function() {
+            var mat, iMat, dist, mod3dObject = [], self = this;
+
+            this.application.ninja.selectedElements.forEach(function(element) {
+                // Reset to the identity matrix
+                //item = this.application.ninja.selectedElements[i];
+                iMat = Matrix.I(4);
+                mat = ElementsMediator.getMatrix(element);
                 mat[12] = 0;
                 mat[13] = 0;
                 mat[14] = 0;
 
-                dist = this._undoArray[i].dist;
-
-                var previousStyleStr = {dist:dist, mat:item.mat};
+                dist = ElementsMediator.getPerspectiveDist(element);
 
-                var newStyleStr = {dist:dist, mat:mat};
+                var previousStyleStr = {dist:dist, mat:element.elementModel.getProperty("mat")};
+                var newStyleStr = {dist:dist, mat:iMat};
 
-                previousStyles.push(previousStyleStr);
-                newStyles.push(newStyleStr);
-            }
+                mod3dObject.push({element:element, properties:newStyleStr, previousProperties: previousStyleStr});
+            });
 
-            ElementsMediator.set3DProperties(this.application.ninja.selectedElements,
-                                            newStyles,
-                                            "Change",
-                                            "translateTool",
-                                            previousStyles
-                                          );
+            ElementsMediator.set3DProperties(mod3dObject, "Change", "rotateTool");
 
                         this.isDrawing = false;
             this.endDraw(event);
 
-//                      this.UpdateSelection(true);
-                        this.Configure(true);
+            // Need to force stage to draw immediately so the new selection center is calculated
+            this.application.ninja.stage.draw();
+            // And captureSelectionDrawn to draw the transform handles
+            this.captureSelectionDrawn(null);
                 }
         },
 
         // We will only translate single elements locally
         _translateLocally: {
                 value: function (transMat) {
+            //console.log( "_translateLocally, startMat: " + this._startMat );
+            //console.log( "_translateLocally, transMat: " + transMat );
+            //console.log( "_translateLocally, startMat: " + this._startMat + ", transMat: " + transMat );
                         var mat = glmat4.multiply(this._startMat, transMat, []);
                         viewUtils.setMatrixForElement( this._target, mat, true );
                         if(this._mode !== 1)
@@ -145,103 +135,81 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
 
         _translateGlobally: {
                 value: function (transMat) {
-                        var len = this._targets.length,
-                                i = 0,
-                                item,
-                                elt,
-                                curMat,
+            //console.log( "_translateGlobally, startMat: " + this._startMat + ", transMat: " + transMat );
+            //console.log( "_translateGlobally, transMat: " + transMat );
+            var self = this,
+                                curMat = viewUtils.getMatrixFromElement( this._target ),
                 matInv = glmat4.inverse(this._startMat, []),
-                nMat = glmat4.multiply(transMat, this._startMat, [] ),
-                            qMat = glmat4.multiply(matInv, nMat, []);
+                nMat = glmat4.multiply(transMat, this._startMat, [] );
+//                          qMat = glmat4.multiply(matInv, nMat, []);
+           
+            if(this._mode === 1) {
+                var curInv = glmat4.inverse( curMat, [] );
+                transMat = glmat4.multiply( nMat, curInv, [] );
+            }
 
                         var shouldUpdateStartMat = true;
 
-                        if(this._clickedOnStage)
-                        {
+                        if(this._clickedOnStage) {
                                 shouldUpdateStartMat = false;
-                        }
-                        else if(this._mode !== 1)
-                        {
+                        } else if(this._mode !== 1) {
                                 this._startMat = nMat;
                         }
 
-                        for(i = 0; i < len; i++)
-                        {
-                                item = this._targets[i];
-                                elt = item.elt;
-                                curMat = item.mat.slice(0);
+            this.application.ninja.selectedElements.forEach(function(element) {
+                curMat = element.elementModel.getProperty("mat").slice(0);
 
-//                              glmat4.multiply(curMat, qMat, curMat);
-//
-//                              viewUtils.setMatrixForElement( elt, curMat, true);
+//                glmat4.multiply(curMat, qMat, curMat);
+//                viewUtils.setMatrixForElement( elt, curMat, true);
                 curMat[12] += transMat[12];
                 curMat[13] += transMat[13];
                 curMat[14] += transMat[14];
-                viewUtils.setMatrixForElement( elt, curMat, true);
 
-                                if(shouldUpdateStartMat)
-                                {
-                                        this._targets[i].mat = curMat;
-                                }
-                        }
+                viewUtils.setMatrixForElement(element, curMat, true);
+
+                if(shouldUpdateStartMat) {
+                    //console.log(  "\t\tshouldUpdateStartMat" );
+                    element.elementModel.setProperty("mat", curMat);
+                }
+
+            });
                 }
         },
 
     _updateTargets: {
-                value: function(addToUndoStack) {
-            var newStyles = [],
-                previousStyles = [],
-                            len = this.application.ninja.selectedElements.length;
-                        this._targets = [];
-                        for(var i = 0; i < len; i++)
-                        {
-                                var elt = this.application.ninja.selectedElements[i];
+                value: function(addToUndo) {
+            var mod3dObject = [], self = this;
 
-                                var curMat = viewUtils.getMatrixFromElement(elt);
-                                var curMatInv = glmat4.inverse(curMat, []);
+            this.application.ninja.selectedElements.forEach(function(element) {
+                if(addToUndo) {
+                    var previousMat = element.elementModel.getProperty("mat").slice(0);
+                    var previousStyleStr = {dist:element.elementModel.getProperty("dist"), mat:MathUtils.scientificToDecimal(previousMat, 5)};
+                    var newStyleStr = {dist:viewUtils.getPerspectiveDistFromElement(element), mat:MathUtils.scientificToDecimal(viewUtils.getMatrixFromElement(element), 5)};
 
-                                this._targets.push({elt:elt, mat:curMat, matInv:curMatInv});
-                if(addToUndoStack)
-                {
-                    var previousStyleStr = {dist:this._undoArray[i].dist, mat:MathUtils.scientificToDecimal(this._undoArray[i].mat.slice(0), 5)};
+                    mod3dObject.push({element:element, properties:newStyleStr, previousProperties: previousStyleStr});
 
-                    var newStyleStr = {dist:viewUtils.getPerspectiveDistFromElement(elt), mat:MathUtils.scientificToDecimal(curMat, 5)};
-
-                    previousStyles.push(previousStyleStr);
-                    newStyles.push(newStyleStr);