4 files changed, 116 insertions, 81 deletions

diff --git a/js/stage/stage.reel/stage.js b/js/stage/stage.reel/stage.js
index 9784f14d..4c1d046b 100755
--- a/js/stage/stage.reel/stage.js
+++ b/js/stage/stage.reel/stage.js
@@ -523,7 +523,7 @@ exports.Stage = Montage.create(Component, {
                 elt = this.application.ninja.currentDocument.GetElementFromPoint(point.x + this.scrollLeft,point.y + this.scrollTop);
 
             // workaround Chrome 3d bug
-            if(this.application.ninja.currentDocument.inExclusion(elt) !== -1)
+            if(this.application.ninja.toolsData.selectedToolInstance._canSnap && this.application.ninja.currentDocument.inExclusion(elt) !== -1)
             {
                 return this._getElementUsingSnapping(point);
             } else {
diff --git a/js/tools/EyedropperTool.js b/js/tools/EyedropperTool.js
index d007da39..11c15158 100755
--- a/js/tools/EyedropperTool.js
+++ b/js/tools/EyedropperTool.js
@@ -18,6 +18,7 @@ exports.EyedropperTool = Montage.create(toolBase, {
     _elementUnderMouse: { value: null },
     _imageDataCanvas: { value: null },
     _imageDataContext: { value: null },
+    _canSnap: { value: false },
 
     Configure: {
         value: function ( doActivate )
@@ -70,7 +71,7 @@ exports.EyedropperTool = Montage.create(toolBase, {
                     this._escape = false;
                 }
 
-                this._updateColor(this._color);
+                this._updateColor(this._color, true);
 
                 this._color = null;
 
@@ -85,18 +86,7 @@ exports.EyedropperTool = Montage.create(toolBase, {
             if(this._color && this._color.value)
             {
                 var color = this.application.ninja.colorController.getColorObjFromCss(this._previousColor);
-
-                if (color && color.value) {
-                    color.value.wasSetByCode = true;
-                    color.value.type = 'change';
-                    if (color.value.a) {
-                        this.application.ninja.colorController.colorModel.alpha = {value: color.value.a,
-                                                                                    wasSetByCode: true,
-                                                                                    type: 'change'};
-                    }
-                    this.application.ninja.colorController.colorModel[color.mode] = color.value;
-                    this._color = null;
-                }
+                this._updateColor(color, true);
             }
             this._escape = true;
         }
@@ -143,24 +133,18 @@ exports.EyedropperTool = Montage.create(toolBase, {
                 {
                     this._deleteImageDataCanvas();
 
-                    c = ElementsMediator.getColor(obj, this._isOverBackground(obj, event));
-                    if(c)
+                    c = this._getColorFromElement(obj, event);
+                    if(typeof(c) === "string")
+                    {
+                        color = this.application.ninja.colorController.getColorObjFromCss(c);
+                    }
+                    else
                     {
                         color = this.application.ninja.colorController.getColorObjFromCss(c.color.css);
                     }
                 }
 
-                if (color && color.value) {
-                    color.value.wasSetByCode = true;
-                    color.value.type = 'changing';
-                    if (color.value.a) {
-                        this.application.ninja.colorController.colorModel.alpha = {value: color.value.a,
-                                                                                    wasSetByCode: true,
-                                                                                    type: 'changing'};
-                    }
-                    this.application.ninja.colorController.colorModel[color.mode] = color.value;
-                    this._color = color;
-                }
+                this._updateColor(color, false);
             }
             else
             {
@@ -172,81 +156,136 @@ exports.EyedropperTool = Montage.create(toolBase, {
     },
 
     _updateColor: {
-        value: function(color) {
-            if (color && color.value) {
-                var input = this.application.ninja.colorController.colorModel.input;
+        value: function(color, updateColorToolBar) {
+            var eventType = "changing";
+            if(updateColorToolBar)
+            {
+                eventType = "change";
+                if (color && color.value)
+                {
+                    var input = this.application.ninja.colorController.colorModel.input;
+
+                    if(input === "fill")
+                    {
+                        this.application.ninja.colorController.colorToolbar.fill_btn.color(color.mode, color.value);
+                    }
+                    else
+                    {
+                        this.application.ninja.colorController.colorToolbar.stroke_btn.color(color.mode, color.value);
+                    }
+
+                    // Updating color chips will set the input type to "chip", so set it back here.
+                    this.application.ninja.colorController.colorModel.input = input;
+                }
+            }
+
+            if(color)
+            {
+                if(color.color)
+                {
+                    color.color.wasSetByCode = true;
+                    color.color.type = eventType;
+                }
 
-                if(input === "fill")
+                if(color.mode === "gradient")
                 {
-                    this.application.ninja.colorController.colorToolbar.fill_btn.color(color.mode, color.value);
+                    this.application.ninja.colorController.colorModel["gradient"] =
+                                    {value: color.color, wasSetByCode: true, type: eventType};
                 }
                 else
                 {
-                    this.application.ninja.colorController.colorToolbar.stroke_btn.color(color.mode, color.value);
+                    if (color.color.a !== undefined)
+                    {
+                        this.application.ninja.colorController.colorModel.alpha =
+                                        {value: color.color.a, wasSetByCode: true, type: eventType};
+                    }
+                    if(color.color.mode)
+                    {
+                        this.application.ninja.colorController.colorModel[color.color.mode] = color.color;
+                    }
+                    else
+                    {
+                        this.application.ninja.colorController.colorModel["rgb"] = color.color;
+                    }
                 }
 
-                // Updating color chips will set the input type to "chip", so set it back here.
-                this.application.ninja.colorController.colorModel.input = input;
-
-                color.value.wasSetByCode = true;
-                color.value.type = 'change';
-                if (color.value.a) {
-                    this.application.ninja.colorController.colorModel.alpha = {value: color.value.a,
-                                                                                wasSetByCode: true,
-                                                                                type: 'change'};
+                if(updateColorToolBar)
+                {
+                    this._previousColor = color.color.css;
+                }
+            }
+            else
+            {
+                this.application.ninja.colorController.colorModel.alpha = {value: 1, wasSetByCode: true, type: eventType};
+                this.application.ninja.colorController.colorModel.applyNoColor();
+                if(updateColorToolBar)
+                {
+                    this._previousColor = "none";
                 }
-                this.application.ninja.colorController.colorModel[color.mode] = color.value;
-                this._previousColor = color.value.css;
             }
+
+            this._color = color;
         }
     },
 
     // TODO - We don't want to calculate this repeatedly
-    _isOverBackground: {
+    _getColorFromElement: {
         value: function(elt, event)
         {
-            var border = ElementsMediator.getProperty(elt, "border", parseFloat);
-
+            var border = ElementsMediator.getProperty(elt, "border"),
+                borderWidth,
+                bounds3D,
+                innerBounds,
+                pt,
+                bt,
+                br,
+                bb,
+                bl,
+                xAdj,
+                yAdj,
+                tmpPt,
+                x,
+                y;
             if(border)
             {
-                var bounds3D,
-                    innerBounds = [],
-                    pt = webkitConvertPointFromPageToNode(this.application.ninja.stage.canvas, new WebKitPoint(event.pageX, event.pageY)),
-                    bt = ElementsMediator.getProperty(elt, "border-top", parseFloat),
-                    br = ElementsMediator.getProperty(elt, "border-right", parseFloat),
-                    bb = ElementsMediator.getProperty(elt, "border-bottom", parseFloat),
-                    bl = ElementsMediator.getProperty(elt, "border-left", parseFloat);
-
-//                this.application.ninja.stage.viewUtils.setViewportObj( elt );
                 bounds3D = this.application.ninja.stage.viewUtils.getElementViewBounds3D( elt );
-//                console.log("bounds");
-//                console.dir(bounds3D);
+                innerBounds = [];
+                pt = webkitConvertPointFromPageToNode(this.application.ninja.stage.canvas,
+                                                                        new WebKitPoint(event.pageX, event.pageY));