Support Quadratic Bézier paths #34
Conversation
Adds support for: Q,q = Quadratic Bézier T,t = Smooth quadratic Bézier
|
Hey @dbworku , thanks for this pull request. I tested it with a couple of SVGs with Question: why did you do those modifications to Cheers Ariel |
|
@arielelkin I created the PocketSVGControlPoint enum (and modified the other methods) because the last control point's validity depends on which curve set the point. 8.3.7 The quadratic Bézier curve commands
8.3.6 The cubic Bézier curve commands
We need to update the control point's state in all methods. However, your question brings up that |
|
Sounds good! |
- Replace check if index == 0 and while statement into `doesValenceMatch` method that produces more descriptive logging when the parameters are insufficient - Compute `[token command] == ` once per append command. - Prevent Q and S control points from mixing - Q and S use last point drawn as control if the previous command did not produce a control point of that type. - Move to point was producing a line after it moved to that point
|
@arielelkin How does it look? |
Support Quadratic Bézier paths
|
Pretty good, thank you! |
Minor README update
Adds support for:
SVG 1.1: Quadratic Bezier Commands
Note: NSBezierPath doesn't support adding quadratic curves. The approximation using the cubic bezier function should be very accurate unless the path is scaled extremely large.