3 files changed, 130 insertions, 94 deletions

diff --git a/js/tools/Translate3DToolBase.js b/js/tools/Translate3DToolBase.js
index 4dca6313..4799b840 100755
--- a/js/tools/Translate3DToolBase.js
+++ b/js/tools/Translate3DToolBase.js
@@ -29,6 +29,8 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
     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,10 +68,6 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
                     delta[0] = 0;
                     delta[1] = 0;
                 }
-                else
-                {
-                    delta[2] = 0;
-                }
                 this._delta = delta.slice(0);
             }
 
@@ -81,6 +79,7 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
             }
             else
             {
+                                console.log( "modifyElements delta: " + delta );
                 this._translateGlobally(transMat);
             }
         }
@@ -134,6 +133,9 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
         // 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,14 +147,23 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
 
         _translateGlobally: {
                 value: function (transMat) {
+                        //console.log( "_translateGlobally, transMat: " + transMat );
+                        //console.log( "_translateGlobally, startMat: " + this._startMat + ", transMat: " + transMat );
                         var len = this._targets.length,
                                 i = 0,
                                 item,
                                 elt,
-                                curMat,
-                matInv = glmat4.inverse(this._startMat, []),
-                nMat = glmat4.multiply(transMat, this._startMat, [] ),
-                            qMat = glmat4.multiply(matInv, nMat, []);
+                                curMat = viewUtils.getMatrixFromElement( this._target ),
+                                matInv = glmat4.inverse(this._startMat, []),
+                                nMat = glmat4.multiply(transMat, this._startMat, [] ),
+                                qMat = glmat4.multiply(matInv, nMat, []);
+                   
+                        if (this._mode === 1)
+                        { 
+                                if (len > 1)  curMat = this._targets[0].mat.slice();
+                                var curInv = glmat4.inverse( curMat, [] );
+                                transMat = glmat4.multiply( nMat, curInv, [] );
+                        }
 
                         var shouldUpdateStartMat = true;
 
@@ -174,24 +185,27 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
 //                              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);
+                                curMat[12] += transMat[12];
+                                curMat[13] += transMat[13];
+                                curMat[14] += transMat[14];
+                                viewUtils.setMatrixForElement( elt, curMat, true);
 
                                 if(shouldUpdateStartMat)
                                 {
+                                        //console.log(  "\t\tshouldUpdateStartMat" );
                                         this._targets[i].mat = curMat;
                                 }
                         }
                 }
         },
 
-    _updateTargets: {
-                value: function(addToUndoStack) {
-            var newStyles = [],
-                previousStyles = [],
-                            len = this.application.ninja.selectedElements.length;
+        _updateTargets: {
+                value: function(addToUndoStack)
+                {
+                        //console.log( "_updateTargets" );
+                        var newStyles = [],
+                                previousStyles = [],
+                                len = this.application.ninja.selectedElements.length;
                         this._targets = [];
                         for(var i = 0; i < len; i++)
                         {
@@ -201,47 +215,47 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
                                 var curMatInv = glmat4.inverse(curMat, []);
 
                                 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)};
+                                if(addToUndoStack)
+                                {
+                                        var previousStyleStr = {dist:this._undoArray[i].dist, mat:MathUtils.scientificToDecimal(this._undoArray[i].mat.slice(0), 5)};
 
-                    var newStyleStr = {dist:viewUtils.getPerspectiveDistFromElement(elt), mat:MathUtils.scientificToDecimal(curMat, 5)};
+                                        var newStyleStr = {dist:viewUtils.getPerspectiveDistFromElement(elt), mat:MathUtils.scientificToDecimal(curMat, 5)};
 
-                    previousStyles.push(previousStyleStr);
-                    newStyles.push(newStyleStr);
-                }
+                                        previousStyles.push(previousStyleStr);
+                                        newStyles.push(newStyleStr);
+                                }
                         }
                         if(addToUndoStack)
                         {
-                ElementsMediator.set3DProperties(this.application.ninja.selectedElements,
-                                                newStyles,
-                                                "Change",
-                                                "translateTool",
-                                                previousStyles
-                                              );
-                if(this._origin && this._delta)
-                {
-                    if(this._handleMode !== null)
-                    {
-                        this._origin[this._handleMode] += this._delta;
-                    }
-                    else
-                    {
-                        this._origin[0] += this._delta[0];
-                        this._origin[1] += this._delta[1];
-                    }
-                }
-                this._delta = null;
-            }
-            // Save previous value for undo/redo
-            this._undoArray = [];
-            for(i = 0, len = this._targets.length; i < len; i++)
-            {
-                var elt = this._targets[i].elt;
-                var _mat = viewUtils.getMatrixFromElement(elt);
-                var _dist = viewUtils.getPerspectiveDistFromElement(elt);
-                this._undoArray.push({mat:_mat, dist:_dist});
-            }
+                                ElementsMediator.set3DProperties(this.application.ninja.selectedElements,
+                                                                                                newStyles,
+                                                                                                "Change",
+                                                                                                "translateTool",
+                                                                                                previousStyles
+                                                                                          );
+                                if(this._origin && this._delta)
+                                {
+                                        if(this._handleMode !== null)
+                                        {
+                                                this._origin[this._handleMode] += this._delta;
+                                        }
+                                        else
+                                        {
+                                                this._origin[0] += this._delta[0];
+                                                this._origin[1] += this._delta[1];
+                                        }
+                                }
+                                this._delta = null;
+                        }
+                        // Save previous value for undo/redo
+                        this._undoArray = [];
+                        for(i = 0, len = this._targets.length; i < len; i++)
+                        {
+                                var elt = this._targets[i].elt;
+                                var _mat = viewUtils.getMatrixFromElement(elt);
+                                var _dist = viewUtils.getPerspectiveDistFromElement(elt);
+                                this._undoArray.push({mat:_mat, dist:_dist});
+                        }
 
                 }
         },
@@ -291,27 +305,27 @@ exports.Translate3DToolBase = Montage.create(ModifierToolBase,
                                 }
                                 else
                                 {
-                    if(this._origin)
-                    {
-                        if(this._delta)
-                        {
-                            if(this._handleMode !== null)
-                            {
+                                        if(this._origin)
+                                        {
+                                                if(this._delta)
+                                                {
+                                                        if(this._handleMode !== null)
+                                                        {
 //                                this._origin[this._handleMode] = this._delta;
-                            }
-                            else
-                            {
+                                                        }
+                                                        else
+                                                        {