Merge pull request #168 from mqg734/EricMerge

Merging Eric's snapping and tool fixes into master.

4 files changed, 110 insertions, 41 deletions

diff --git a/js/helper-classes/3D/draw-utils.js b/js/helper-classes/3D/draw-utils.js
index ec8eea0f..7bd77c3a 100755
--- a/js/helper-classes/3D/draw-utils.js
+++ b/js/helper-classes/3D/draw-utils.js
@@ -803,20 +803,21 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                 context.stroke();
 
                                 this._selectionCtr = MathUtils.getCenterFromBounds(3, bounds3D);
-                                console.log("selection center, single elt case - ", this._selectionCtr);
+//                              console.log("selection center, single elt case - ", this._selectionCtr);
                                 
                                 this.viewUtils.popViewportObj();
                         }
                         else
                         {
                                 // get the plane from the first element to compare against the other elements
+                                var dot;
                                 var flat = true;
-                                var plane = this.viewUtils.getElementPlane( eltArray[0] );
+                                var plane = this.viewUtils.getUnprojectedElementPlane( eltArray[0] );
                                 for (i=1;  i<len;  i++)
                                 {
                                         elt = eltArray[i];
-                                        var plane2 = this.viewUtils.getElementPlane( elt );
-                                        var dot = MathUtils.dot3(plane,plane2);
+                                        var plane2 = this.viewUtils.getUnprojectedElementPlane( elt );
+                                        dot = Math.abs( MathUtils.dot3(plane,plane2) );
                                         if (MathUtils.fpCmp(dot, 1) != 0)
                                         {
                                                 flat = false;
@@ -832,6 +833,19 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                         }
                                 }
 
+                                // if all of the planes are aligned, check if they are aligned with the view direction
+                                if (flat)
+                                {
+                                        var stage = this.application.ninja.currentDocument.documentRoot;
+                                        var stageMat = this.viewUtils.getMatrixFromElement(stage);
+                                        var viewDir = [ stageMat[8], stageMat[9], stageMat[10] ];
+                                        viewDir = vecUtils.vecNormalize( 3, viewDir );
+                                        dot = Math.abs( MathUtils.dot3(plane,viewDir) );
+                                        if (MathUtils.fpCmp(dot, 1) != 0)
+                                                flat = false;
+                                }
+//                              console.log( "drawSelectionBounds, flat: " + flat );
+
                                 // if all the elements share the same plane, draw the 2D rectangle
                                 if (flat)
                                 {
@@ -843,12 +857,12 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
 
                                                 // get the element bounds in 'plane' space
                                                 bounds = this.viewUtils.getElementViewBounds3D( elt );
-                        ssMat = this.viewUtils.getLocalToGlobalMatrix( elt );
+                                                ssMat = this.viewUtils.getLocalToGlobalMatrix( elt );
                                                 for (j=0;  j<4;  j++)
                                                 {
-                            var localPt = bounds[j];
-                            tmpPt = this.viewUtils.localToGlobal2(localPt, ssMat);
-                            pt = tmpPt;
+                                                        var localPt = bounds[j];
+                                                        tmpPt = this.viewUtils.localToGlobal2(localPt, ssMat);
+                                                        pt = tmpPt;
 
                                                         if (!rect)
                                                         {
@@ -865,7 +879,7 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                         // draw the rectangle
                                         context.beginPath();
 
-                    pt = MathUtils.makeDimension3(rect.getPoint(3));
+                                        pt = MathUtils.makeDimension3(rect.getPoint(3));
 
                                         bounds3D = [[0,0], [0,0], [0,0], [0,0]];
                                         this._selectionCtr = pt.slice(0);
@@ -873,7 +887,7 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                         context.moveTo( pt[0],  pt[1] );
                                         for (i=0;  i<4;  i++)
                                         {
-                        pt = rect.getPoint(i);
+                                                pt = rect.getPoint(i);
                                                 context.lineTo( pt[0],  pt[1] );
                                                 bounds3D[i] = pt.slice(0);
                                         }
@@ -890,19 +904,21 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                 {
                                         var minPt,  maxPt;
 
-                    // we set the root to "the world".
-                    var saveRoot = this.viewUtils.getRootElement();
-                    this.viewUtils.setRootElement( this.viewUtils.getStageElement() );
-                    ssMat = this.viewUtils.getLocalToGlobalMatrix( this._sourceSpaceElt );
+                                        // we set the root to "the world".
+                                        var saveRoot = this.viewUtils.getRootElement();
+                                        this.viewUtils.setRootElement( this.viewUtils.getStageElement() );
+                                        ssMat = this.viewUtils.getLocalToGlobalMatrix( this._sourceSpaceElt );
 
                                         for (i=0;  i<len;  i++)
                                         {
                                                 elt = eltArray[i];
                                                 bounds = this.viewUtils.getElementViewBounds3D( elt );
-                        var eltMat = this.viewUtils.getLocalToGlobalMatrix( elt );
+                                                var eltMat = this.viewUtils.getLocalToGlobalMatrix( elt );
                                                 for (j=0;  j<4;  j++)
                                                 {
                                                         pt = this.viewUtils.localToGlobal2( bounds[j],  eltMat );
+                                                        var tmpPt = this.viewUtils.localToStageWorld(bounds[j], elt);
+                                                        tmpPt = this.viewUtils.screenToView( tmpPt[0], tmpPt[1], tmpPt[2] );
                                                         if (!minPt)
                                                         {
                                                                 minPt = pt.slice(0);
@@ -939,7 +955,7 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
 
                                         // get the 8 corners of the parallelpiped in world space
                                         var wc = new Array();   // wc == world cube
-                    wc.push(  this.viewUtils.localToGlobal2( [x0,y0,z1], ssMat ) );
+                                        wc.push(  this.viewUtils.localToGlobal2( [x0,y0,z1], ssMat ) );
                                         wc.push(  this.viewUtils.localToGlobal2( [x0,y1,z1], ssMat ) );
                                         wc.push(  this.viewUtils.localToGlobal2( [x1,y1,z1], ssMat ) );
                                         wc.push(  this.viewUtils.localToGlobal2( [x1,y0,z1], ssMat ) );
@@ -970,7 +986,7 @@ var DrawUtils = exports.DrawUtils = Montage.create(Component, {
                                         //context.strokeStyle = ((right > 0) || (back > 0)) ? dark : light;  context.beginPath();
                                         if ((right > 0) || (back > 0)) {
                                                 context.beginPath();
-                        p = this.viewUtils.localToGlobal2( [x1, y0, z0], ssMat );  context.moveTo( p[0], p[1] );
+                                                p = this.viewUtils.localToGlobal2( [x1, y0, z0], ssMat );  context.moveTo( p[0], p[1] );
                                                 p = this.viewUtils.localToGlobal2( [x1, y1, z0], ssMat );  context.lineTo( p[0], p[1] );
                                                 context.closePath();  context.stroke();
                                         }
diff --git a/js/helper-classes/3D/hit-record.js b/js/helper-classes/3D/hit-record.js
index 265bf2a4..d72e33bd 100755
--- a/js/helper-classes/3D/hit-record.js
+++ b/js/helper-classes/3D/hit-record.js
@@ -292,6 +292,59 @@ var HitRecord = exports.HitRecord = Object.create(Object.prototype,
 
             return str;
         }
-    }
+    },
+
+        test:
+        {
+                value: function()
+                {
+                        var elt = this.getElement();
+                        var stage = viewUtils.getStage();
+                        if (elt === stage)  return;
+
+                        var localPt = this.calculateElementPreTransformScreenPoint();
+                        var stageWorldPt = this.calculateStageWorldPoint();
+                        var globalPt = this.getScreenPoint();
+                        var err = false;
+
+                        var test1 = viewUtils.localToGlobal( localPt, elt );
+                        var dist = vecUtils.vecDist(3, test1, globalPt);
+                        if (MathUtils.fpSign(dist) != 0)
+                        {
+                                err = true;
+                                console.log( "**** transform error 1 ***** " + dist + ", localPt: " + localPt );
+                        }
+
+                        var stageWorldToGlobal = viewUtils.getStageWorldToGlobalMatrix();
+                        var test2 = MathUtils.transformAndDivideHomogeneousPoint( stageWorldPt, stageWorldToGlobal );
+                        dist = vecUtils.vecDist(3, test2, globalPt);
+                        if (MathUtils.fpSign(dist) != 0)
+                        {
+                                err = true;
+                                console.log( "**** transform error 2 ***** " + dist + ", localPt: " + localPt );
+                        }
+
+                        var localToGlobal = viewUtils.getLocalToGlobalMatrix( elt );
+                        var globalToLocal = glmat4.inverse( localToGlobal, [] );
+                        var test3 = MathUtils.transformAndDivideHomogeneousPoint( globalPt, globalToLocal );
+                        dist = vecUtils.vecDist(3, test3, localPt);