fix path exports svgs (fabricjs#4274)

FlowMSP · Sep 6, 2017 · d267843 · d267843
1 parent c46fa74
commit d267843
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/gradient.class.js b/src/gradient.class.js
@@ -173,18 +173,22 @@
     toSVG: function(object) {
       var coords = clone(this.coords, true), i, len,
           markup, commonAttributes, colorStops = clone(this.colorStops, true),
-          needsSwap = coords.r1 > coords.r2;
+          needsSwap = coords.r1 > coords.r2,
+          offsetX = object.width / 2, offsetY = object.height / 2;
       // colorStops must be sorted ascending
       colorStops.sort(function(a, b) {
         return a.offset - b.offset;
       });
-
+      if (object.type === 'path') {
+        offsetX = 0;
+        offsetY = 0;
+      }
       for (var prop in coords) {
         if (prop === 'x1' || prop === 'x2') {
-          coords[prop] += this.offsetX - object.width / 2;
+          coords[prop] += this.offsetX - offsetX;
         }
         else if (prop === 'y1' || prop === 'y2') {
-          coords[prop] += this.offsetY - object.height / 2;
+          coords[prop] += this.offsetY - offsetY;
         }
       }
 

diff --git a/src/shapes/ellipse.class.js b/src/shapes/ellipse.class.js
@@ -116,10 +116,10 @@
      * @return {String} svg representation of an instance
      */
     toSVG: function(reviver) {
-      var markup = this._createBaseSVGMarkup(), x = 0, y = 0;
+      var markup = this._createBaseSVGMarkup();
       markup.push(
         '<ellipse ', this.getSvgId(),
-          'cx="', x, '" cy="', y, '" ',
+          'cx="0" cy="0" ',
           'rx="', this.rx,
           '" ry="', this.ry,
           '" style="', this.getSvgStyles(),