11 files changed, 330 insertions, 215 deletions

diff --git a/js/document/html-document.js b/js/document/html-document.js
index 3876670c..454b7154 100755
--- a/js/document/html-document.js
+++ b/js/document/html-document.js
@@ -207,16 +207,16 @@ exports.HTMLDocument = Montage.create(TextDocument, {
                         var elt = this.documentRoot;
                         if (elt)
                         {
+                                /*
+                                // Use this code to test the runtime version of WebGL
+                                var cdm = new NinjaCvsRt.CanvasDataManager();
+                                cdm.loadGLData(elt,  value,  null );
+                                */
+
+                                // /*
                                 var nWorlds= value.length;
                                 for (var i=0;  i<nWorlds;  i++)
                                 {
-                                        /*
-                                        // Use this code to test the runtime version of WebGL
-                                        var cdm = new NinjaCvsRt.CanvasDataManager();
-                                        cdm.loadGLData(elt,  value,  null );
-                                        */
-
-                                        // /*
                                         // get the data for the next canvas
                                         var importStr = value[i];
 
@@ -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
@@ -281,8 +281,8 @@ exports.HTMLDocument = Montage.create(TextDocument, {
                                                         }
                                                 }
                                         }
-                                        // */
                                 }
+                                // */
                         }
                 }
     },
diff --git a/js/helper-classes/3D/snap-manager.js b/js/helper-classes/3D/snap-manager.js
index cd8cc102..7fc492a5 100755
--- a/js/helper-classes/3D/snap-manager.js
+++ b/js/helper-classes/3D/snap-manager.js
@@ -948,7 +948,7 @@ var SnapManager = exports.SnapManager = Montage.create(Component, {
                                                         {
 
                                                                 // 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)
@@ -1011,7 +1011,8 @@ var SnapManager = exports.SnapManager = Montage.create(Component, {
             }
                         // hit test the current object
                         var hit;
-                        if (depth > 0)  // don't snap to the root
+                        var snapToStage = ((depth === 0) && (elt === this.application.ninja.currentSelectedContainer) && (elt.nodeName === 'CANVAS'));
+                        if ((depth > 0) || snapToStage) // don't snap to the root unles we are working inside a canvas
                         {
                                 // if the element is in the 2D cache snapping is done there
                                 if (elt.elementModel && !elt.elementModel.isIn2DSnapCache)
@@ -1077,7 +1078,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 +1321,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 );
-
+                                                rtnVal = true;
+                                        }
+                                }
+                        }