Merge branch 'refs/heads/ninja-internal-master' into cut-copy-paste

1 files changed, 24 insertions, 52 deletions

diff --git a/js/components/gradientpicker.reel/gradientpicker.js b/js/components/gradientpicker.reel/gradientpicker.js
index b10f6624..7fd5d918 100755
--- a/js/components/gradientpicker.reel/gradientpicker.js
+++ b/js/components/gradientpicker.reel/gradientpicker.js
@@ -6,54 +6,43 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 
 ////////////////////////////////////////////////////////////////////////
 //
-var Montage =                   require("montage/core/core").Montage,
-        Component =             require("montage/ui/component").Component;
+var Montage =   require("montage/core/core").Montage,
+        Component = require("montage/ui/component").Component;
 ////////////////////////////////////////////////////////////////////////
 //Exporting as ColorWheel
 exports.GradientPicker = Montage.create(Component, {
         ////////////////////////////////////////////////////////////////////
         //
         hasTemplate: {
-                enumerable: true,
         value: true
     },
     ////////////////////////////////////////////////////////////////////
     //
     _updating: {
-        enumerable: false,
         value: false
     },
     ////////////////////////////////////////////////////////////////////
     //
     _value: {
-        enumerable: false,
         value: null
     },
         ////////////////////////////////////////////////////////////////////
     //
     value: {
-        enumerable: true,
-        get: function() {
-            return this._value;
-        },
-        set: function(value) {
-            this._value = value;
-        }
+        get: function() {return this._value;},
+        set: function(value) {this._value = value;}
     },
     ////////////////////////////////////////////////////////////////////
     //
     _mode: {
-        enumerable: false,
         value: 'linear'
     },
         ////////////////////////////////////////////////////////////////////
     //
     mode: {
-        enumerable: true,
-        get: function() {
-            return this._mode;
-        },
+        get: function() {return this._mode;},
         set: function(value) {
+                //
                 this.application.ninja.colorController.colorPopupManager.hideColorChipPopup();
                 //
             this._mode = value;
@@ -64,7 +53,6 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     prepareForDraw: {
-        enumerable: false,
         value: function() {
                 //
                 }
@@ -72,23 +60,21 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     willDraw: {
-        enumerable: false,
         value: function() {
                 //Getting component views from layout
-                this.element._components = {trackCover: this.element.getElementsByClassName('cp_gp_slider')[0].getElementsByClassName('cover')[0], gradientTrack: this.element.getElementsByClassName('cp_gp_slider')[0].getElementsByClassName('track')[0], stopsContainer: this.element.getElementsByClassName('cp_gp_slider')[0].getElementsByClassName('chips')[0]};
-                this.element._trackWidth = parseInt(getComputedStyle(this.element._components.stopsContainer).getPropertyCSSValue('width').cssText);
+                this.element._trackWidth = parseInt(getComputedStyle(this.trackChips).getPropertyCSSValue('width').cssText);
                 //TODO: Fix events and remove this hack
-                this.element._components.trackCover.addEventListener('mouseover', function () {
+                this.trackCover.addEventListener('mouseover', function () {
                                 if (!this._updating) {                  
-                                this.element._components.trackCover.style.display = 'none';
+                                this.trackCover.style.display = 'none';
                         }
                 }.bind(this), true);
                 //
-                this.element.getElementsByClassName('cp_gp_linear')[0].addEventListener('change', function (e){
+                this.radioLinear.addEventListener('change', function (e){
                         this.mode = 'linear';
                 }.bind(this), true);
                 //
-                this.element.getElementsByClassName('cp_gp_radial')[0].addEventListener('change', function (e){
+                this.radioRadial.addEventListener('change', function (e){
                         this.mode = 'radial';
                 }.bind(this), true);    
                 }
@@ -96,22 +82,19 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     draw: {
-        enumerable: false,
         value: function() {
                 //
                 if (this.mode === 'linear') {
-                        this.element.getElementsByClassName('cp_gp_linear')[0].checked = 'true';
+                        this.radioLinear.checked = 'true';
                 } else if (this.mode === 'radial') {
-                        this.element.getElementsByClassName('cp_gp_radial')[0].checked = 'true';
+                        this.radioRadial.checked = 'true';
                 }
                 //
                 if (!this.value) {
                         this.addDefaultStops();
                         } else {
-                                //Temp holder
-                                var stops = this.value;
                                 //Adding stops from preset value
-                                for (var i=0; stops[i]; i++) {
+                                for (var i=0, stops = this.value; stops[i]; i++) {
                                         this.addStop({color: {mode: stops[i].mode, value: stops[i].value}, percent:stops[i].position}, true);
                                 }
                         }
@@ -120,10 +103,9 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
         didDraw: {
-        enumerable: false,
         value: function() {
                         //
-                        this.element._components.gradientTrack.addEventListener('click', this, false);
+                        this.trackMain.addEventListener('click', this, false);
                 
                 
                 
@@ -132,7 +114,7 @@ exports.GradientPicker = Montage.create(Component, {
                 ////////////////////////////////////////////////////////////
                         ////////////////////////////////////////////////////////////
                         //TODO: Determing a better way to get screen position
-                        var element = this.element._components.gradientTrack;
+                        var element = this.trackMain;
                 this.element._trackX = 0, this.element._trackY = 0;
                 //
                         while (element && !isNaN(element.offsetLeft) && !isNaN(element.offsetTop)) {
@@ -152,7 +134,6 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     addDefaultStops: {
-        enumerable: false,
         value: function() {
                 this.addStop({color: {mode: 'rgb', value: {r: 255, g: 255, b: 255, a: 1, css: 'rgb(255, 255, 255)'}}, percent: 0}, true);
                         this.addStop({color: {mode: 'rgb', value: {r: 0, g: 0, b: 0, a: 1, css: 'rgb(0, 0, 0)'}}, percent: 100}, true);
@@ -161,7 +142,6 @@ exports.GradientPicker = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////