2 files changed, 182 insertions, 77 deletions

diff --git a/js/panels/color/colorpanelpopup.reel/colorpanelpopup.js b/js/panels/color/colorpanelpopup.reel/colorpanelpopup.js
index 153817c1..179dc02c 100755
--- a/js/panels/color/colorpanelpopup.reel/colorpanelpopup.js
+++ b/js/panels/color/colorpanelpopup.reel/colorpanelpopup.js
@@ -60,7 +60,7 @@ exports.ColorPanelPopup = Montage.create(Component, {
         value: function () {
                 //
                 this._components = null;
-                this._components = {wheel: null, combo: null};
+                this._components = {wheel: null, combo: null, gradient: null};
         }
     },
     ////////////////////////////////////////////////////////////////////
@@ -369,27 +369,30 @@ exports.ColorPanelPopup = Montage.create(Component, {
     drawGradient: {
         value: function (g) {
                 //TODO: Remove container, insert in reel
-                var container = document.createElement('div'), gradient = GradientPicker.create();
+                var container = document.createElement('div');
+                //
+                this._components.gradient = GradientPicker.create();
+                //
                 this.gradients.appendChild(container);
                 //Creating gradient picker from components
-                gradient.element = container;
-                gradient.hack = this.hack; // TODO: Remove
+                this._components.gradient.element = container;
+                this._components.gradient.hack = this.hack; // TODO: Remove
                 //
                 if (g && g.value && g.value.stops) {
                         if (g.value.gradientMode) {
-                                gradient._mode = g.value.gradientMode;
-                                gradient.value = g.value.stops;
+                                this._components.gradient._mode = g.value.gradientMode;
+                                this._components.gradient.value = g.value.stops;
                         } else {
-                                gradient._mode = 'linear';
-                                gradient.value = g.value.stops;
+                                this._components.gradient._mode = 'linear';
+                                this._components.gradient.value = g.value.stops;
                         }
                 } else {
-                        gradient._mode = this.defaultGradient.gradientMode;
-                        gradient.value = this.defaultGradient.stops;
+                        this._components.gradient._mode = this.defaultGradient.gradientMode;
+                        this._components.gradient.value = this.defaultGradient.stops;
                 }
                 //
-                gradient.needsDraw = true;
-                gradient.addEventListener('change', function (e){
+                this._components.gradient.needsDraw = true;
+                this._components.gradient.addEventListener('change', function (e){
                         //
                         if (!e._event.wasSetByCode) {
                                 this.colorManager.gradient = {value: e._event.gradient, type: 'change', wasSetByCode: false};
diff --git a/js/panels/color/colorpopup-manager.js b/js/panels/color/colorpopup-manager.js
index bccfcebc..717e6fa2 100755
--- a/js/panels/color/colorpopup-manager.js
+++ b/js/panels/color/colorpopup-manager.js
@@ -36,18 +36,30 @@ exports.ColorPopupManager = Montage.create(Component, {
             //Closing popups if outside limits
             document.addEventListener('mousedown', function (e) {
                 //
-                if (this._popupBase && !this._popupChipBase) {
+                if (this._popupBase && !this._popupChipBase && !this._popupGradientChipBase) {
                         if(!this.popupHitCheck(this._popupBase, e)) {
                                 this.hideColorPopup();
                         }
-                } else if (!this._popupBase && this._popupChipBase) {
+                } else if (!this._popupBase && this._popupChipBase && !this._popupGradientChipBase) {
                         if(!this.popupHitCheck(this._popupChipBase, e)) {
                                 this.hideColorChipPopup();
                         }
-                } else if (this._popupBase && this._popupChipBase) {
+                } else if (this._popupBase && this._popupChipBase && !this._popupGradientChipBase) {
                         if(!this.popupHitCheck(this._popupBase, e) && !this.popupHitCheck(this._popupChipBase, e)) {
                                 this.hideColorPopup();
                         }
+                } else if (this._popupBase && this._popupChipBase && this._popupGradientChipBase) {
+                        if(!this.popupHitCheck(this._popupBase, e) && !this.popupHitCheck(this._popupChipBase, e) && !this.popupHitCheck(this._popupGradientChipBase, e)) {
+                                this.hideColorPopup();
+                        }
+                } else if (!this._popupBase && this._popupChipBase && this._popupGradientChipBase) {
+                        if(!this.popupHitCheck(this._popupChipBase, e) && !this.popupHitCheck(this._popupGradientChipBase, e)) {
+                                this.hideColorChipPopup();
+                        }
+                } else if (this._popupBase && !this._popupChipBase && this._popupGradientChipBase) {
+                        if(!this.popupHitCheck(this._popupBase, e) && !this.popupHitCheck(this._popupGradientChipBase, e)) {
+                                this.hideColorPopup();
+                        }
                 }
             }.bind(this));
             ////////////////////////////////////////////////////////////
@@ -74,9 +86,11 @@ exports.ColorPopupManager = Montage.create(Component, {
                         //Hides popups since click is outside limits
                         return false;
                 } else {
+                        //Keeps popup open since click inside area
                         return true;
                 }
             } else {
+                //Hides popups since element not detected
                     return false;
             }
                 }  
@@ -92,6 +106,11 @@ exports.ColorPopupManager = Montage.create(Component, {
         value: null
     },
     ////////////////////////////////////////////////////////////////////
+    //
+    _popupGradientChipBase: {
+        value: null
+    },
+    ////////////////////////////////////////////////////////////////////
     //Storing color manager
     _colorManager: {
         value: null
@@ -99,33 +118,33 @@ exports.ColorPopupManager = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     colorManager: {
-        get: function() {
-            return this._colorManager;
-        },
-        set: function(value) {
-                this._colorManager = value;
-        }
+        get: function() {return this._colorManager;},
+        set: function(value) {this._colorManager = value;}
     },
     ////////////////////////////////////////////////////////////////////
     //
     _colorPopupDrawing: {
-        enumerable: true,
         value: false
     },
     ////////////////////////////////////////////////////////////////////
     //
         _colorChipPopupDrawing: {
-        enumerable: true,
+        value: false
+    },
+    ////////////////////////////////////////////////////////////////////
+    //
+        _colorGradientPopupDrawing: {
         value: false
     },
     ////////////////////////////////////////////////////////////////////
     //TODO: Remove and use montage's built in component
     showColorPopup: {
-        enumerable: true,
         value: function (x, y, side, align) {
+                //Check to see if popup is drawing, avoids errors
                 if (this._colorPopupDrawing) {
                         return;
                 }
+                //Check for toggling view
                 if (this._popupBase && this._popupBase.opened) {
                         //Toogles if called and opened
                         this.hideColorPopup();
@@ -138,7 +157,7 @@ exports.ColorPopupManager = Montage.create(Component, {
                                 this._hasinit = true;
                         }
                         ////////////////////////////////////////////////////
-                        //Creating popup from m-js component
+                        //Creating popup
                         var popup = document.createElement('div');
                         document.body.appendChild(popup);
                         //
@@ -158,13 +177,13 @@ exports.ColorPopupManager = Montage.create(Component, {
     ////////////////////////////////////////////////////////////////////
     //
     hideColorPopup: {
-        enumerable: true,
         value: function () {
                 if (this._popupBase && this._popupBase.opened) {
                         //
                         this._popupBase.popup.removeEventListener('didDraw', this, false);
                         //
                         this.hideColorChipPopup();
+                        this.hideGradientChipPopup();