In SVGPathEditor, an error was found in path editing of cubic curve s…

…egments 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.
dsward2 · Apr 4, 2017 · 6cd1243 · 6cd1243
1 parent 68d4d2b
commit 6cd1243
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/macSVG/MacSVG-Info.plist b/macSVG/MacSVG-Info.plist
@@ -105,7 +105,7 @@
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>0649</string>
+	<string>0652</string>
 	<key>LSApplicationCategoryType</key>
 	<string>public.app-category.graphics-design</string>
 	<key>LSMinimumSystemVersion</key>

diff --git 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