Fix arcto sweep flag when a transformation matrix is applied

When a transformation matrix flips the coordinates horizontally or vertically (but not both) clockwise arcs need to become counterclockwise and vice-versa. Fixes #708
svg · Jun 30, 2017 · 40bcb7f · 40bcb7f
1 parent bc3c8a6
commit 40bcb7f
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/plugins/_transforms.js b/plugins/_transforms.js
@@ -283,6 +283,12 @@ exports.transformArc = function(arc, transform) {
         arc[2] = ((major ? term2 < 0 : term1 > 0) ? -1 : 1) *
             Math.acos((major ? term1 : term2) / Math.sqrt(term1 * term1 + term2 * term2)) * 180 / Math.PI;
     }
+
+    if ((transform[0] < 0) !== (transform[3] < 0)) {
+        // Flip the sweep flag if coordinates are being flipped horizontally XOR vertically
+        arc[4] = 1 - arc[4];
+    }
+
     return arc;
 
 };

diff --git a/test/plugins/convertPathData.18.svg b/test/plugins/convertPathData.18.svg