From a11ef2eed7049835c8bdfa50a2b893632c46eaa0 Mon Sep 17 00:00:00 2001
From: Valerio Virgillito
Date: Wed, 4 Apr 2012 11:11:58 -0700
Subject: Squashed commit of Preparing for the montage undo-manager:
 Architecture changes

Reworked the add and remove elements into 1 function which can take 1 or more elements.
Removed the _element from the selection array
Many other changes related to those 2 changes
Undo/Redo shortcuts are now using montage undo/redo manager.

Signed-off-by: Valerio Virgillito <valerio@motorola.com>
---
 js/tools/Rotate3DToolBase.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'js/tools/Rotate3DToolBase.js')

diff --git a/js/tools/Rotate3DToolBase.js b/js/tools/Rotate3DToolBase.js
index b04e8b0a..eb2cdba4 100755
--- a/js/tools/Rotate3DToolBase.js
+++ b/js/tools/Rotate3DToolBase.js
@@ -262,7 +262,7 @@ exports.Rotate3DToolBase = Montage.create(ModifierToolBase, {
             {
                 if(len === 1)
                 {
-                    this.target = this.application.ninja.selectedElements[0]._element;
+                    this.target = this.application.ninja.selectedElements[0];
                     drawUtils.addElement(this.target);
 
                     viewUtils.pushViewportObj( this.target );
@@ -311,7 +311,7 @@ exports.Rotate3DToolBase = Montage.create(ModifierToolBase, {
             this._targets = [];
             for(var i = 0; i < len; i++)
             {
-                var elt = this.application.ninja.selectedElements[i]._element;
+                var elt = this.application.ninja.selectedElements[i];
 //                this._initProps3D(elt);
 
 
-- 
cgit v1.2.3


From 1e2e0a8d04e55c0ecfb0b1b674792b69fb091415 Mon Sep 17 00:00:00 2001
From: Nivesh Rajbhandari
Date: Thu, 5 Apr 2012 12:01:38 -0700
Subject: Fixing 3d tool reset to redraw handles.

Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>
---
 js/tools/Rotate3DToolBase.js | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

(limited to 'js/tools/Rotate3DToolBase.js')

diff --git a/js/tools/Rotate3DToolBase.js b/js/tools/Rotate3DToolBase.js
index eb2cdba4..696408bf 100755
--- a/js/tools/Rotate3DToolBase.js
+++ b/js/tools/Rotate3DToolBase.js
@@ -438,24 +438,23 @@ exports.Rotate3DToolBase = Montage.create(ModifierToolBase, {
         value : function()
         {
             var item,
-                elt,
                 mat,
                 dist,
                 newStyles = [],
                 previousStyles = [],
-                len = this._targets.length,
+                len = this.application.ninja.selectedElements.length,
                 iMat;
             for(var i = 0; i < len; i++)
             {
                 // Reset to the identity matrix
-                item = this._targets[i];
+                item = this.application.ninja.selectedElements[i];
                 iMat = Matrix.I(4);
-                mat = item.mat;
+                mat = ElementsMediator.getMatrix(item);
 //                iMat[12] = mat[12];
 //                iMat[13] = mat[13];
 //                iMat[14] = mat[14];
 
-                dist = this._undoArray[i].dist;
+                dist = ElementsMediator.getPerspectiveDist(item);
 
                 var previousStyleStr = {dist:dist, mat:mat};
 
@@ -476,8 +475,10 @@ exports.Rotate3DToolBase = Montage.create(ModifierToolBase, {
             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);
         }
     },
 
-- 
cgit v1.2.3