Creating shapes in a container

3 files changed, 15 insertions, 2 deletions

diff --git a/js/tools/SelectionTool.js b/js/tools/SelectionTool.js
index 7033ee3d..9c9aee93 100755
--- a/js/tools/SelectionTool.js
+++ b/js/tools/SelectionTool.js
@@ -222,8 +222,10 @@ var SelectionTool = exports.SelectionTool = Montage.create(ModifierToolBase, {
         value: function(event) {
             if(this.application.ninja.selectedElements.length > 0) {
                 this.application.ninja.currentSelectedContainer = this.application.ninja.selectedElements[0];
+                                console.log( "setting container to selected element" );
             } else {
                 this.application.ninja.currentSelectedContainer = this.application.ninja.currentDocument.documentRoot;
+                                console.log( "setting container to stage" );
             }
         }
     },
diff --git a/js/tools/ShapeTool.js b/js/tools/ShapeTool.js
index 74e692b6..90b403a4 100755
--- a/js/tools/ShapeTool.js
+++ b/js/tools/ShapeTool.js
@@ -64,6 +64,7 @@ exports.ShapeTool = Montage.create(DrawingTool, {
             if(drawData) {
                 var canvas;
                 if(!this._useExistingCanvas()) {
+                                        console.log( "creating a new canvas" );
                     canvas = NJUtils.makeNJElement("canvas", "Canvas", "shape", {"data-RDGE-id": NJUtils.generateRandom()}, true);
                     var elementModel = TagTool.makeElement(~~drawData.width, ~~drawData.height,
                                                                         drawData.planeMat, drawData.midPt, canvas, true);
@@ -71,6 +72,7 @@ exports.ShapeTool = Montage.create(DrawingTool, {
                     canvas.elementModel.isShape = true;
                                         this.application.ninja.elementMediator.addElements(canvas, elementModel.data);
                 } else {
+                                        console.log( "inserting shape into container" );
                     canvas = this._targetedElement;
                     if (!canvas.getAttribute( "data-RDGE-id" ))
                         canvas.setAttribute( "data-RDGE-id", NJUtils.generateRandom() );
diff --git a/js/tools/drawing-tool-base.js b/js/tools/drawing-tool-base.js
index 2283dfab..0b51aa28 100755
--- a/js/tools/drawing-tool-base.js
+++ b/js/tools/drawing-tool-base.js
@@ -41,11 +41,20 @@ exports.DrawingToolBase = Montage.create(Montage, {
 
                         // do the snap
                         var hitRec = snapManager.snap(x, y,  true);
+                        var dragPlane;
                         if (hitRec) {
                                 // set up the working plane and convert the hit record to be working plane relative
-                                var dragPlane = snapManager.setupDragPlanes( hitRec );
-                                var wpHitRec = hitRec.convertToWorkingPlane( dragPlane );
+                                if (hitRec.getElement() === snapManager.application.ninja.currentSelectedContainer)
+                                {
+                                        dragPlane = viewUtils.getUnprojectedElementPlane( hitRec.getElement() );
+                                        snapManager.setupDragPlaneFromPlane( dragPlane );
+                                }
+                                else
+                                {
+                                        dragPlane = snapManager.setupDragPlanes( hitRec );
+                                }
 
+                                var wpHitRec = hitRec.convertToWorkingPlane( dragPlane );
                                 var pt = hitRec.getScreenPoint();
 
                 return( [wpHitRec, pt[0], pt[1]] );