From 730c42e005b1ebd96448e3bc2dd700ffd8909cb5 Mon Sep 17 00:00:00 2001
From: Pushkar Joshi
Date: Tue, 13 Mar 2012 08:22:06 -0700
Subject: turn off the fill color for pen tool paths by default

---
 js/lib/geom/sub-path.js | 2 +-
 js/tools/PenTool.js     | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

(limited to 'js')

diff --git a/js/lib/geom/sub-path.js b/js/lib/geom/sub-path.js
index 0a65511b..017cca92 100755
--- a/js/lib/geom/sub-path.js
+++ b/js/lib/geom/sub-path.js
@@ -73,7 +73,7 @@ var GLSubpath = function GLSubpath() {
     this._materialAmbient = [0.2, 0.2, 0.2, 1.0];
     this._materialDiffuse = [0.4, 0.4, 0.4, 1.0];
     this._materialSpecular = [0.4, 0.4, 0.4, 1.0];
-    this._fillColor = [0.4, 0.4, 0.4, 1.0];
+    this._fillColor = [1.0, 1.0, 1.0, 0.0];
     this._fillMaterial = null;
     this._DISPLAY_ANCHOR_RADIUS = 5;
     //drawing context
diff --git a/js/tools/PenTool.js b/js/tools/PenTool.js
index 6897c003..60ef4f92 100755
--- a/js/tools/PenTool.js
+++ b/js/tools/PenTool.js
@@ -467,9 +467,9 @@ exports.PenTool = Montage.create(ShapeTool, {
                     if (this.application.ninja.colorController.colorToolbar.stroke.webGlColor){
                         this._selectedSubpath.setStrokeColor(this.application.ninja.colorController.colorToolbar.stroke.webGlColor);
                     }
-                    if (this.application.ninja.colorController.colorToolbar.fill.webGlColor){
-                        this._selectedSubpath.setFillColor(this.application.ninja.colorController.colorToolbar.fill.webGlColor);
-                    }
+                    //if (this.application.ninja.colorController.colorToolbar.fill.webGlColor){
+                    //    this._selectedSubpath.setFillColor(this.application.ninja.colorController.colorToolbar.fill.webGlColor);
+                    //}
                 } //if this is a new path being rendered
 
                 this._selectedSubpath.makeDirty();
-- 
cgit v1.2.3