From 6cd12437de9a2c3d25ebd06665f5fb952718a832 Mon Sep 17 00:00:00 2001 From: dsward2 Date: Mon, 3 Apr 2017 20:04:31 -0500 Subject: [PATCH] In SVGPathEditor, an error was found in path editing of cubic curve segments where extra editing handles were added improperly for the "x0y0" and "x3y3" implied control points (reflections of "x1y1" and "x2y2", respectively). The extra handles did not function correctly, particularly for the curve continuity option. The x0y0 reflected handle should be drawn when the previous segment was a Moveto ("M" or "m") command. The x3y3 reflected handle should be drawn for the last segment of a cubic curve. --- macSVG/MacSVG-Info.plist | 2 +- macSVG/SVGDocument Classes/SVGPathEditor.m | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/macSVG/MacSVG-Info.plist b/macSVG/MacSVG-Info.plist index 7fa9616..7599ed1 100644 --- a/macSVG/MacSVG-Info.plist +++ b/macSVG/MacSVG-Info.plist @@ -105,7 +105,7 @@ CFBundleSignature ???? CFBundleVersion - 0649 + 0652 LSApplicationCategoryType public.app-category.graphics-design LSMinimumSystemVersion diff --git a/macSVG/SVGDocument Classes/SVGPathEditor.m b/macSVG/SVGDocument Classes/SVGPathEditor.m index 0f34d25..a17ac82 100644 --- a/macSVG/SVGDocument Classes/SVGPathEditor.m +++ b/macSVG/SVGDocument Classes/SVGPathEditor.m @@ -2303,14 +2303,14 @@ -(void) makePathHandles if (segmentIdx >= (pathSegmentsCount - 1)) { - reflectX1Y1 = YES; + //reflectX1Y1 = YES; reflectX2Y2 = YES; } if ((previousPathCommand == 'M') || (previousPathCommand == 'm')) { reflectX1Y1 = YES; - reflectX2Y2 = YES; + //reflectX2Y2 = YES; } [self addHandleForCubicCurveto:pathSegmentDictionary segmentIndex:segmentIdx