Merge branch 'refs/heads/master' into montage-v10-integration

Conflicts:
	js/io/system/ninjalibrary.json

Signed-off-by: Valerio Virgillito <valerio@motorola.com>

4 files changed, 75 insertions, 21 deletions

diff --git a/assets/canvas-runtime.js b/assets/canvas-runtime.js
index fe5f839c..feb35187 100644
--- a/assets/canvas-runtime.js
+++ b/assets/canvas-runtime.js
@@ -2107,7 +2107,40 @@ NinjaCvsRt.RuntimeBrushStroke = Object.create(NinjaCvsRt.RuntimeGeomObj, {
                     }
                     this._LocalPoints = newPoints.slice(0);
                 }
-            }
+
+                // *** compute the bounding box *********
+                var BBoxMin = [Infinity, Infinity, Infinity];
+                var BBoxMax = [-Infinity, -Infinity, -Infinity];
+                if (numPoints === 0) {
+                    BBoxMin = [0, 0, 0];
+                    BBoxMax = [0, 0, 0];
+                } else {
+                    for (var i=0;i<numPoints;i++){
+                        var pt = this._LocalPoints[i];
+                        for (var d = 0; d < 3; d++) {
+                            if (BBoxMin[d] > pt[d]) {
+                                BBoxMin[d] = pt[d];
+                            }
+                            if (BBoxMax[d] < pt[d]) {
+                                BBoxMax[d] = pt[d];
+                            }
+                        }//for every dimension d from 0 to 2
+                    }
+                }
+
+                //increase the bbox given the stroke width and the angle (in case of calligraphic brush)
+                var bboxPadding = this._strokeWidth*0.5;
+                for (var d = 0; d < 3; d++) {
+                    BBoxMin[d]-= bboxPadding;
+                    BBoxMax[d]+= bboxPadding;
+                }//for every dimension d from 0 to 2
+
+                //******* update the local coords so that the bbox min is at the origin ******
+                for (var i=0;i<numPoints;i++) {
+                    this._LocalPoints[i][0]-= BBoxMin[0];
+                    this._LocalPoints[i][1]-= BBoxMin[1];
+                }
+            }//if we need to do smoothing
         }
     },
 
@@ -2231,7 +2264,10 @@ NinjaCvsRt.RuntimeBrushStroke = Object.create(NinjaCvsRt.RuntimeGeomObj, {
                     var disp = [brushStamp[t][0], brushStamp[t][1]];
                     var alphaVal = 1.0;
                     var distFromOpaqueRegion = Math.abs(t-halfNumTraces) - opaqueRegionHalfWidth;
-                    if (distFromOpaqueRegion>0) {
+                    if (numTraces === 1){
+                        distFromOpaqueRegion = 0;
+                    }
+                    else if (distFromOpaqueRegion>0) {
                         var transparencyFactor = distFromOpaqueRegion/maxTransparentRegionHalfWidth;
                         alphaVal = 1.0 - transparencyFactor;//(transparencyFactor*transparencyFactor);//the square term produces nonlinearly varying alpha values
                         alphaVal *= 0.5; //factor that accounts for lineWidth == 2
diff --git a/js/io/system/ninjalibrary.json b/js/io/system/ninjalibrary.json
index e38dd147..fb108937 100644
--- a/js/io/system/ninjalibrary.json
+++ b/js/io/system/ninjalibrary.json
@@ -1,6 +1,6 @@
 {
         "libraries": [
                 {"name": "Montage", "path": "/node_modules/descriptor.json", "version": "0.10.1.0"},
-                {"name": "RDGE", "path": "/assets/descriptor.json", "version": "0.5.9.0"}
+                {"name": "RDGE", "path": "/assets/descriptor.json", "version": "0.6.0.0"}
         ]
 }
\ No newline at end of file
diff --git a/js/lib/geom/brush-stroke.js b/js/lib/geom/brush-stroke.js
index d9c2ab53..0e0406dd 100755
--- a/js/lib/geom/brush-stroke.js
+++ b/js/lib/geom/brush-stroke.js
@@ -719,7 +719,10 @@ BrushStroke.prototype.drawToContext = function(ctx, drawStageWorldPts, stageWorl
             var disp = [brushStamp[t][0], brushStamp[t][1]];
             var alphaVal = 1.0;
             var distFromOpaqueRegion = Math.abs(t-halfNumTraces) - opaqueRegionHalfWidth;
-            if (distFromOpaqueRegion>0) {
+            if (numTraces === 1){
+                distFromOpaqueRegion = 0;
+            }
+            else if (distFromOpaqueRegion>0) {
                 var transparencyFactor = distFromOpaqueRegion/maxTransparentRegionHalfWidth;
                 alphaVal = 1.0 - transparencyFactor;//(transparencyFactor*transparencyFactor);//the square term produces nonlinearly varying alpha values
                 alphaVal *= 0.5; //factor that accounts for lineWidth == 2
diff --git a/js/tools/PenTool.js b/js/tools/PenTool.js
index e296d0e5..33ca5c71 100755
--- a/js/tools/PenTool.js
+++ b/js/tools/PenTool.js
@@ -288,16 +288,35 @@ exports.PenTool = Montage.create(ShapeTool, {
 
             //assume we are not starting a new path as we will set this to true if we create a new Subpath()
             this._isNewPath = false;
-            
-            //if we had closed the selected subpath previously, or if we have not yet started anything, create a subpath
-            if (this._entryEditMode !== this.ENTRY_SELECT_PATH && this._selectedSubpath && this._selectedSubpath.getIsClosed() && this._makeMultipleSubpaths) {
-                this._selectedSubpath = null;
-            }
 
             if (this._subtool !== this.SUBTOOL_NONE && this._selectedSubpath===null) {
                 //do nothing because the pen plus and pen minus subtools need a selected subpath
                 return;
             }
+
+            //build the hit record for the current mouse position (on the stage or the plane of the path canvas)
+            var hitRec = this.getHitRecord(event.pageX, event.pageY, false);
+            var globalMousePos=null, localMousePos=null, stageWorldMousePos = null, drawingCanvas=null;
+            if (this._selectedSubpathCanvas){
+                globalMousePos = hitRec.getScreenPoint();
+                localMousePos = ViewUtils.globalToLocal(globalMousePos, this._selectedSubpathCanvas);
+            }
+
+            //if we have a selected subpath and canvas, check if the current click location hits the path
+            // todo optimize this...currently pickPath is called twice (see later in this function)
+            var hitPath = false;
+            if (this._selectedSubpath && this._selectedSubpathCanvas ) {
+                var tempSelAnchorAndParamAndCode = this._selectedSubpath.pickPath(localMousePos[0], localMousePos[1], localMousePos[2], this._PICK_POINT_RADIUS, true);
+                if (tempSelAnchorAndParamAndCode[2] !== this._selectedSubpath.SEL_NONE){     // ******* if we hit the path anywhere *********
+                    hitPath = true;
+                }
+            }
+
+            //if we had closed the selected subpath previously, or if we have not yet started anything, create a subpath
+            if (this._entryEditMode !== this.ENTRY_SELECT_PATH && this._selectedSubpath && this._selectedSubpath.getIsClosed() && this._makeMultipleSubpaths && !hitPath) {
+                this._selectedSubpath = null;
+            }
+
             if (this._selectedSubpath === null) {
                 this._selectedSubpath = new SubPath();
                 this._selectedSubpathCanvas = null;
@@ -335,9 +354,7 @@ exports.PenTool = Montage.create(ShapeTool, {
                 this._selectedSubpath.setFillColor(colorArray);
             } //if the selectedSubpath was null and needed to be constructed
 
-            //build the hit record for the current mouse position (on the stage or the plane of the path canvas)
-            var hitRec = this.getHitRecord(event.pageX, event.pageY, false);
-            var globalMousePos=null, localMousePos=null, stageWorldMousePos = null, drawingCanvas=null;
+            //build the current mouse position in stage world space in case we don't already have a canvas
             if (!this._selectedSubpathCanvas){
                 drawingCanvas = this.application.ninja.currentDocument.model.documentRoot;//ViewUtils.getStageElement();
                 stageWorldMousePos = hitRec.calculateStageWorldPoint();
@@ -345,10 +362,7 @@ exports.PenTool = Montage.create(ShapeTool, {
                 stageWorldMousePos[1]+= snapManager.getStageHeight()*0.5;
                 localMousePos = stageWorldMousePos; //since the subpath points are in stage world space, set the 'localMousePos' to be stage world as well
             }
-            else {
-                globalMousePos = hitRec.getScreenPoint();
-                localMousePos = ViewUtils.globalToLocal(globalMousePos, this._selectedSubpathCanvas);
-            }
+
 
             if (this._selectedSubpathCanvas === null && this._subtool===this.SUBTOOL_NONE){
                 //IF this is the first anchor point of the selected subpath
@@ -358,7 +372,6 @@ exports.PenTool = Montage.create(ShapeTool, {
                 if (this._selectedSubpath.getNumAnchors()===0) {
                     this._selectedSubpathPlaneMat   = hitRec.getPlaneMatrix();
                 }
-
                 //check if the mouse click location is close to the existing anchor
                 var indexAndCode = this._selectedSubpath.pickAnchor(stageWorldMousePos[0], stageWorldMousePos[1], stageWorldMousePos[2], this._PICK_POINT_RADIUS);
                 if (indexAndCode[0]>=0){
@@ -526,7 +539,9 @@ exports.PenTool = Montage.create(ShapeTool, {
             
             //set the cursor to be the default cursor (depending on whether the selected subpath has any points yet)
             if (this._subtool===this.SUBTOOL_NONE){
-                if (this._selectedSubpath && this._selectedSubpath.getNumAnchors()>0){
+                if ((this._selectedSubpath && this._selectedSubpath.getNumAnchors()>0 && !this._selectedSubpath.getIsClosed())
+                    ||
+                    this._entryEditMode === this.ENTRY_SELECT_PATH){
                     this.application.ninja.stage.drawingCanvas.style.cursor = //"auto";
                         "url('images/cursors/penCursors/Pen_.png') 5 1, default";
                 }
@@ -853,9 +868,9 @@ exports.PenTool = Montage.create(ShapeTool, {
             this._editMode = this.EDIT_NONE;
 
             //if we're not in edit_path mode and we closed the selected subpath, then we are going to start a new subpath, so we nullify the selected subpath
-            if (this._selectedSubpath && this._selectedSubpath.getIsClosed() && this._entryEditMode !== this.ENTRY_SELECT_PATH){
-                this._selectedSubpath = null;
-            }
+            //if (this._selectedSubpath && this._selectedSubpath.getIsClosed() && this._entryEditMode !== this.ENTRY_SELECT_PATH){
+            //    this._selectedSubpath = null;
+            //}
             
             if (this._selectedSubpath){
                 this.DrawSubpathAnchors(this._selectedSubpath);//render the subpath anchors on canvas