7 files changed, 161 insertions, 195 deletions

diff --git a/js/document/html-document.js b/js/document/html-document.js
index 2531465d..cce88747 100755
--- a/js/document/html-document.js
+++ b/js/document/html-document.js
@@ -228,7 +228,7 @@ exports.HTMLDocument = Montage.create(TextDocument, {
                                         {
                                                 // JSON format.  pull off the
                                                 importStr = importStr.substr( index+1 );
-                                                jObj = jObj = JSON.parse( importStr );
+                                                jObj = JSON.parse( importStr );
                                                 id = jObj.id;
                                         }
                                         else
diff --git a/js/helper-classes/3D/snap-manager.js b/js/helper-classes/3D/snap-manager.js
index cd8cc102..2c975985 100755
--- a/js/helper-classes/3D/snap-manager.js
+++ b/js/helper-classes/3D/snap-manager.js
@@ -1077,7 +1077,7 @@ var SnapManager = exports.SnapManager = Montage.create(Component, {
                         var hitRec = this.snapToScreenBounds( elt, globalScrPt, bounds, bounds3D );
 
                         // see if we can snap to a contained geometry object
-                        if (hitRec && this.getGLWorld(elt) && !this.isARectangle(elt))
+                        if (hitRec && this.getGLWorld(elt))     // && !this.isARectangle(elt))
                         {
                                 var localPt = hitRec.calculateElementWorldPoint();
                                 if (hitRec.getType() != hitRec.SNAP_TYPE_ELEMENT)
@@ -1320,193 +1320,135 @@ var SnapManager = exports.SnapManager = Montage.create(Component, {
                 }
         },
 
-        snapToContainedElement : 
+        doSnapToContainedElement:
         {
                 value: function( eyePt,  dir,  glObj,  hitRec, targetScrPt ) 
                 {
                         var rtnVal = false;
-                        var elt = hitRec.getElement();
 
+                        var elt = hitRec.getElt();
                         var world = glObj.getWorld();
-                        switch (glObj.geomType())
+
+                        var nearVrt = glObj.getNearVertex( eyePt, dir );
+                        if (nearVrt)
                         {
-                                case glObj.GEOM_TYPE_RECTANGLE:
-                                        break;
+                                var viewPt = this.GLToView(nearVrt, world );
+                                var mat = viewUtils.getMatrixFromElement( elt );
+                                var worldPt = MathUtils.transformPoint( viewPt, mat );
 
-                                case glObj.GEOM_TYPE_CIRCLE:
+                                viewUtils.pushViewportObj( elt );
+                                var scrPt = viewUtils.viewToScreen( worldPt );
+                                var offset = viewUtils.getElementOffset( elt );
+                                MathUtils.makeDimension3( offset );
+                                var parentPt = vecUtils.vecAdd(3, scrPt, offset );
+                                var globalPt = viewUtils.localToGlobal( parentPt, elt.offsetParent );
+
+                                var dist = vecUtils.vecDist(2, globalPt, targetScrPt );
+                                if (dist < this.ELEMENT_VERTEX_HIT_RAD)
+                                {
+                                        console.log( "hit a vertex" );
+
+                                        // check if the distance is less than
+                                        // the distance on the current hit record
+                                        if (dist <= vecUtils.vecDist(2, targetScrPt, hitRec.getScreenPoint() ))
                                         {
-                                                var nearVrt = glObj.getNearVertex( eyePt, dir );
-                                                if (nearVrt)
-                                                {
-                                                        var viewPt = this.GLToView(nearVrt, world );
-                                                        var mat = viewUtils.getMatrixFromElement( elt );
-                                                        var worldPt = MathUtils.transformPoint( viewPt, mat );
-
-                                                        viewUtils.pushViewportObj( elt );
-                                                        var scrPt = viewUtils.viewToScreen( worldPt );
-                                                        var offset = viewUtils.getElementOffset( elt );
-                                                        MathUtils.makeDimension3( offset );
-                                                        var parentPt = vecUtils.vecAdd(3, scrPt, offset );
-                                                        var globalPt = viewUtils.localToGlobal( parentPt, elt.offsetParent );
-
-                                                        var dist = vecUtils.vecDist(2, globalPt, targetScrPt );
-                                                        if (dist < this.ELEMENT_VERTEX_HIT_RAD)
-                                                        {
-                                                                // check if the distance is less than
-                                                                // the distance on the current hit record
-                                                                if (dist <= vecUtils.vecDist(2, targetScrPt, hitRec.getScreenPoint() ))
-                                                                {
-                                                                        hitRec.setScreenPoint( globalPt );
-                                                                        //var localMatInv = hitRec.getPlaneMatrix().inverse();
-                                                                        var localMatInv = glmat4.inverse( hitRec.getPlaneMatrix(), []);
-                                                                        viewUtils.pushViewportObj( hitRec.getElement() );
-                                                                        var localPt = viewUtils.screenToView( scrPt[0], scrPt[1], scrPt[2] );
-                                                                        viewUtils.popViewportObj();
-                                                                        localPt = MathUtils.transformPoint( localPt, localMatInv );
-                                                                        hitRec.setLocalPoint( localPt );
-                                                                        hitRec.setType( hitRec.SNAP_TYPE_CONTAINED_ELEMENT );
-
-                                                                        rtnVal = true;
-                                                                }
-                                                        }
-                                                }       // if (nearVrt)
+                                                console.log( "rejected - further than existing snap" );
+
+                                                hitRec.setScreenPoint( globalPt );
+                                                //var localMatInv = hitRec.getPlaneMatrix().inverse();
+                                                var localMatInv = glmat4.inverse( hitRec.getPlaneMatrix(), []);
+                                                viewUtils.pushViewportObj( hitRec.getElement() );
+                                                var localPt = viewUtils.screenToView( scrPt[0], scrPt[1], scrPt[2] );
+                                                viewUtils.popViewportObj();
+                                                localPt = MathUtils.transformPoint( localPt, localMatInv );
+                                                hitRec.setLocalPoint( localPt );
+                                                hitRec.setType( hitRec.SNAP_TYPE_CONTAINED_ELEMENT );
 
-                                                if (!rtnVal)
-                                                {
-                                                        var nearPt = glObj.getNearPoint( eyePt,  dir );
-                                                        if (nearPt)
-                                                        {
-                                                                var viewPt = this.GLToView(nearPt, world );
-                                                                var mat = viewUtils.getMatrixFromElement( elt );
-                                                                var worldPt = MathUtils.transformPoint( viewPt, mat );
-
-                                                                viewUtils.pushViewportObj( elt );
-                                                                var scrPt = viewUtils.viewToScreen( worldPt );
-                                                                var offset = viewUtils.getElementOffset( elt );
-                                                                MathUtils.makeDimension3( offset );
-                                                                var parentPt = vecUtils.vecAdd(3, scrPt, offset );
-                                                                var globalPt = viewUtils.localToGlobal( parentPt, elt.offsetParent );
-
-                                                                var dist = vecUtils.vecDist(2, globalPt, targetScrPt );
-                                                                if (dist < this.ELEMENT_EDGE_HIT_RAD)
-                                                                {
-                                                                        // check if the distance is less than
-                                                                        // the distance on the current hit record
-                                                                        //var dist2 = vecUtils.vecDist(2, targetScrPt, hitRec.getScreenPoint() );
-                                                                        //if (dist <= dist2+1 )
-                                                                        {
-                                                                                hitRec.setScreenPoint( globalPt );
-                                                                                //var localMatInv = hitRec.getPlaneMatrix().inverse();
-                                                                                var localMatInv = glmat4.inverse( hitRec.getPlaneMatrix(), []);
-                                                                                viewUtils.pushViewportObj( hitRec.getElement() );
-                                                                                var localPt = viewUtils.screenToView( scrPt[0], scrPt[1], scrPt[2] );
-                                                                                viewUtils.popViewportObj();
-                                                                                localPt = MathUtils.transformPoint( localPt, localMatInv );
-                                                                                hitRec.setLocalPoint( localPt );
-                                                                                hitRec.setType( hitRec.SNAP_TYPE_CONTAINED_ELEMENT );
-
-                                                                                rtnVal = true;
-                                                                        }
-                                                                }
-                                                        }
-                                                }       // if (!rtnVal)
+                                                rtnVal = true;
+                                        }
+                                }
+                        }
 
-                                                if (!rtnVal && glObj.containsPoint( eyePt,  dir ))
+                        if (!rtnVal)
+                        {
+                                var nearPt = glObj.getNearPoint( eyePt,  dir );
+                                if (nearPt)
+                                {
+                                        var viewPt = this.GLToView(nearPt, world );
+                                        var mat = viewUtils.getMatrixFromElement( elt );
+                                        var worldPt = MathUtils.transformPoint( viewPt, mat );
+
+                                        viewUtils.pushViewportObj( elt );
+                                        var scrPt = viewUtils.viewToScreen( worldPt );
+                                        var offset = viewUtils.getElementOffset( elt );
+                                        MathUtils.makeDimension3( offset );
+                                        var parentPt = vecUtils.vecAdd(3, scrPt, offset );
+                                        var globalPt = viewUtils.localToGlobal( parentPt, elt.offsetParent );
+
+                                        var dist = vecUtils.vecDist(2, globalPt, targetScrPt );
+                                        if (dist < this.ELEMENT_EDGE_HIT_RAD)
+                                        {
+                                                console.log( "hit an edge" );
+
+                                                // check if the distance is less than
+                                                // the distance on the current hit record
+                                                //var dist2 = vecUtils.vecDist(2, targetScrPt, hitRec.getScreenPoint() );
+                                                //if (dist <= dist2+1 )
                                                 {
+                                                        hitRec.setScreenPoint( globalPt );
+                                                        //var localMatInv = hitRec.getPlaneMatrix().inverse();
+                                                        var localMatInv = glmat4.inverse( hitRec.getPlaneMatrix(), []);
+                                                        viewUtils.pushViewportObj( hitRec.getElement() );
+                                                        var localPt = viewUtils.screenToView( scrPt[0], scrPt[1], scrPt[2] );