New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix spikes from wide strokes (#6924) #6928
Conversation
Cool ! This PR also fixed #6903 . |
Thank you for this PR! Great explanation. Will review it. ping @SukantPal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bringing these changes! In the future, I think we'll need to do work reducing the technical debt in this file.
Thanks for the quick responses! @bigtimebuddy, I recommend reviewing the change with whitespace ignored – it makes much more sense that way: https://github.com/pixijs/pixi.js/pull/6928/files?diff=unified&w=1. @SukantPal, I agree that |
Codecov Report
@@ Coverage Diff @@
## dev #6928 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 15 15
Lines 671 671
=========================================
Hits 671 671 Continue to review full report at Codecov.
|
Another good one @joshuahhh, you're on a roll! Thanks. |
Description of change
This fixes #6924, the spikes-behind-wide-joins problem.
Before: (miter, bevel, round)
After: (miter, bevel, round)
It's not perfect. In particular, some regions are drawn twice, so strokes with alpha will be a bit mis-drawn. But that was the case before anyway. I believe some documentation already recommends using filters rather than stroke alpha in such cases.
(Note that some edges of the triangulation are incorrectly drawn red in the "round" examples above. This is an unrelated bug, which you can see occurs in both the "before" and "after" cases. If I can isolate it, I will report it as a separate issue.)
Pre-Merge Checklist
npm run lint
)npm run test
)Thanks so much!