Merge pull request #454 from plotly/illustrator-compatibility

Illustrator compatibility

Author: mdtusz

src/plots/geo/geo.js

@@ -308,10 +308,7 @@ proto.adjustLayout = function(geoLayout, graphSize) {
             width: geoLayout._width,
             height: geoLayout._height
         })
-        .style({
-            'fill': geoLayout.bgcolor,
-            'stroke-width': 0
-        });
+        .call(Color.fill, geoLayout.bgcolor);
 };
 
 proto.drawTopo = function(selection, layerName, geoLayout) {

src/snapshot/tosvg.js

@@ -105,6 +105,11 @@ module.exports = function toSVG(gd, format) {
         }
     }
 
+    // remove draglayer for Adobe Illustrator compatibility
+    if(fullLayout._draggers) {
+        fullLayout._draggers.node().remove();
+    }
+
     // in case the svg element had an explicit background color, remove this
     // we want the rect to get the color so it's the right size; svg bg will
     // fill whatever container it's displayed in regardless of plot size.

test/image/baselines/geo_bg-color.png

@@ -145,4 +145,34 @@ describe('Test color:', function() {
             expect(container2).toEqual(expectedContainer2);
         });
     });
+
+    describe('fill', function() {
+
+        it('should call style with both fill and fill-opacity', function() {
+            var mockElement = {
+                style: function(object) {
+                    expect(object.fill).toBe('rgb(255, 255, 0)');
+                    expect(object['fill-opacity']).toBe(0.5);
+                }
+            };
+
+            Color.fill(mockElement, 'rgba(255,255,0,0.5');
+        });
+
+    });
+
+    describe('stroke', function() {
+
+        it('should call style with both fill and fill-opacity', function() {
+            var mockElement = {
+                style: function(object) {
+                    expect(object.stroke).toBe('rgb(255, 255, 0)');
+                    expect(object['stroke-opacity']).toBe(0.5);
+                }
+            };
+
+            Color.stroke(mockElement, 'rgba(255,255,0,0.5');
+        });
+
+    });
 });