From d5aec31ee519069ff6120e5d5685828750501e67 Mon Sep 17 00:00:00 2001
From: Jose Antonio Marquez
Date: Sat, 16 Jun 2012 19:04:13 -0700
Subject: More cleaning up

Controller will need a review and refactoring.
---
 js/controllers/color-controller.js | 48 +++++++-------------------------------
 js/panels/color/colorpanel.js      | 13 -----------
 2 files changed, 8 insertions(+), 53 deletions(-)

diff --git a/js/controllers/color-controller.js b/js/controllers/color-controller.js
index 3edaff86..3165988d 100755
--- a/js/controllers/color-controller.js
+++ b/js/controllers/color-controller.js
@@ -19,13 +19,11 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     hasTemplate: {
-        enumerable: true,
         value: false
     },
     ////////////////////////////////////////////////////////////////////
     //
     deserializedFromTemplate: {
-    	enumerable: true,
     	value: function () {
     		//Setting up colorManager in other classes
     		this.colorPanelBase.colorManager = this.colorModel;
@@ -38,60 +36,47 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     colorModel: {
-        enumerable: true,
         value: ColorModel
     },
     ////////////////////////////////////////////////////////////////////
     //
     colorPanelBase: {
-        enumerable: true,
         value: ColorPanelBase
     },
     ////////////////////////////////////////////////////////////////////
     //
     colorPopupManager: {
-        enumerable: true,
         value: ColorPopupManager
     },
     ////////////////////////////////////////////////////////////////////
     //
     colorButtonManager: {
-        enumerable: true,
         value: ColorPopupManager
     },
     ////////////////////////////////////////////////////////////////////
     //
     colorView: {
-        enumerable: true,
         value: null
     },
     ////////////////////////////////////////////////////////////////////
     //
     colorToolbar: {
-        enumerable: true,
         value: null
     },
     ////////////////////////////////////////////////////////////////////
     //
     _popupTab: {
-    	enumerable: false,
     	value: 'wheel'
     },
     ////////////////////////////////////////////////////////////////////
     //
     popupTab: {
-    	enumerable: true,
-        get: function() {
-            return this._popupTab;
-        },
-        set: function(value) {
-        	this._popupTab = value.toLowerCase();
-        }
+        get: function() {return this._popupTab;},
+        set: function(value) {this._popupTab = value.toLowerCase();}
     },
     ////////////////////////////////////////////////////////////////////
     //
     addButton: {
-        enumerable: true,
         value: function (type, button) {
             if (this.colorView) {
                 this.colorView.addButton(type, button);
@@ -107,7 +92,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     removeButton: {
-        enumerable: true,
         value: function (type, button) {
             if (this.colorView) {
                 this.colorView.removeButton(type, button);
@@ -123,41 +107,28 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     _fill: {
-    	enumerable: false,
     	value: null
     },
     ////////////////////////////////////////////////////////////////////
     //
     fill: {
-    	enumerable: true,
-        get: function() {
-            return this._fill;
-        },
-        set: function(value) {
-        	this._fill = value;
-        }
+        get: function() {return this._fill;},
+        set: function(value) {this._fill = value;}
     },
     ////////////////////////////////////////////////////////////////////
     //
     _stroke: {
-    	enumerable: false,
     	value: null
     },
     ////////////////////////////////////////////////////////////////////
     //
     stroke: {
-    	enumerable: true,
-        get: function() {
-            return this._stroke;
-        },
-        set: function(value) {
-        	this._stroke = value;
-        }
+        get: function() {return this._stroke;},
+        set: function(value) {this._stroke = value;}
     },
     ////////////////////////////////////////////////////////////////////
     //
     getBackground: {
-        enumerable: true,
         value: function (element) {
         	//TODO: Return object with all background properties
         	console.log(ElementsMediator.getProperty(element, 'background-color'));
@@ -167,7 +138,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     getBorder: {
-        enumerable: true,
         value: function (element) {
         	
         }
@@ -175,7 +145,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     setColor: {
-        enumerable: true,
         value: function (mode, color, isFill, selection) {
             var elements;
             //The selection is optional, if none, it asks for the currently selected elements
@@ -195,7 +164,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     handleChange: {
-    	enumerable: true,
         value: function (e) {
         	//
             var color, input = e._event.input, panelMode, mode = e._event.mode;
@@ -244,7 +212,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //Converts CSS to a color object to be used by the color model
     getColorObjFromCss: {
-    	enumerable: true,
         value: function (css) {
         	//
         	var color, arr, i, j, temp, c, gradient;
@@ -341,7 +308,6 @@ exports.ColorController = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //Parses simple solid CSS string into color object
     parseCssToColor: {
-    	enumerable: true,
         value: function (css) {
         	var color, r, p;
         	//Parsing string and converting into color object
@@ -368,4 +334,6 @@ exports.ColorController = Montage.create(Component, {
        		return color;
         }
     }
+    ////////////////////////////////////////////////////////////////////
+    ////////////////////////////////////////////////////////////////////
 });
\ No newline at end of file
diff --git a/js/panels/color/colorpanel.js b/js/panels/color/colorpanel.js
index 3a603e98..0985c717 100755
--- a/js/panels/color/colorpanel.js
+++ b/js/panels/color/colorpanel.js
@@ -33,21 +33,8 @@ exports.ColorPanel = Montage.create(PanelBase, {
             this.content = this.application.ninja.colorController.colorView = this.application.ninja.colorController.colorPanelBase.create();
             //Checking for first draw to apply default colors
             this.content.addEventListener('firstDraw', this, false);
-
-
-            ////////////////////////////////////////////////////////////
-            ////////////////////////////////////////////////////////////
-            //TODO: Remove and add via toolbar draw loop
-
-            ////////////////////////////////////////////////////////////
-            ////////////////////////////////////////////////////////////
-
-
         }
     },
-    ////////////////////////////////////////////////////////////////////
-    //Applying default colors only on first draw
-
     ////////////////////////////////////////////////////////////////////
     ////////////////////////////////////////////////////////////////////
 });
\ No newline at end of file
-- 
cgit v1.2.3