Updated ink bottle tool to use new setColor routine in the mediator and controllers.

Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>

2 files changed, 25 insertions, 13 deletions

diff --git a/js/tools/FillTool.js b/js/tools/FillTool.js
index fabcec69..46d76a19 100644
--- a/js/tools/FillTool.js
+++ b/js/tools/FillTool.js
@@ -63,15 +63,14 @@ exports.FillTool = Montage.create(ModifierToolBase, {
                     colorInfo = { mode:color.colorMode,
                                        color:color.color
                                     };
-                    ElementsMediator.setColor(this.application.ninja.selectedElements, colorInfo, true, "Change", "fillTool");
                 }
                 else
                 {
                     colorInfo = { mode:"nocolor",
                                        color:color.color
                                     };
-                    ElementsMediator.setColor(this.application.ninja.selectedElements, colorInfo, true, "Change", "fillTool");
                 }
+                ElementsMediator.setColor(this.application.ninja.selectedElements, colorInfo, true, "Change", "fillTool");
             }
         }
     }
diff --git a/js/tools/InkBottleTool.js b/js/tools/InkBottleTool.js
index cc20d94c..2be74ada 100644
--- a/js/tools/InkBottleTool.js
+++ b/js/tools/InkBottleTool.js
@@ -48,20 +48,33 @@ exports.InkBottleTool = Montage.create(ModifierToolBase, {
         value: function(event) {
             this.isDrawing = true;
 
-            if(this._canColor)
+            if(this._canColor && this.application.ninja.selectedElements.length)
             {
-//                var color = this.application.ninja.colorController.colorToolbar.stroke;
-//                ElementsMediator.setColor(this.application.ninja.selectedElements, color, false, "Change", "inkBottleTool");
+                var color = this.application.ninja.colorController.colorToolbar.stroke,
+                    colorInfo;
+                if(color && color.color)
+                {
+                    colorInfo = { mode:color.colorMode,
+                                   color:color.color
+                                };
+                }
+                else
+                {
+                    colorInfo = { mode:"nocolor",
+                                   color:color.color
+                                };
+                }
+
+                colorInfo.borderInfo = { borderStyle:this.options._borderStyle.value,
+                                         borderWidth:this.options._borderWidth.value,
+                                         borderUnits:this.options._borderWidth.units
+                                       };
 
-                var strokeInfo = {  borderStyle:this.options._borderStyle.value,
-                                    borderWidth:this.options._borderWidth.value,
-                                    borderUnits:this.options._borderWidth.units,
-                                    strokeSize:this.options._strokeSize.value,
-                                    strokeUnits:this.options._strokeSize.units,
-                                    color:this.application.ninja.colorController.colorToolbar.stroke
-                                 };
+                colorInfo.strokeInfo = { strokeSize:this.options._strokeSize.value,
+                                         strokeUnits:this.options._strokeSize.units
+                                       };
 
-                ElementsMediator.setStroke(this.application.ninja.selectedElements, strokeInfo, "Change", "inkBottleTool");
+                ElementsMediator.setColor(this.application.ninja.selectedElements, colorInfo, false, "Change", "inkBottleTool");
             }
         }
     }