Add fill tool support on strokes#2664
Add fill tool support on strokes#2664seam0s-dev wants to merge 40 commits intoGraphiteEditor:masterfrom
Conversation
|
Test cases: |
9741537 to
1a64e29
Compare
|
See the video for the issues this encounters: capture_10_.mp4Here's the file with this shape (remove the |
|
@Keavon The issue does not seem to affect shapes constructed using pen tool: ellipse_vs_path_shapes_fill_overlay.mp4 |
|
You likely need to take into account the local transform, the nested transform spaces to reach from local space to document space, and the stroke transform (which is affected by scaling or skewing a grouped layer, for example). |
…ty while showing overlays
7f66924 to
535fba8
Compare
|
I have force pushed this branch to move it to a new base. Ensure you have the branch checked out. Then please run |
316a737 to
532e913
Compare
ec51271 to
e025103
Compare
44f9e60 to
f1f2ffe
Compare
This PR adds support for fill on strokes while using the fill tool as requested in #2615.