Add transform support to bezier shapes in clip() by VANSH3104 · Pull Request #8481 · processing/p5.js
Navigation Menu
{{ message }}
processing / p5.js Public
- Notifications You must be signed in to change notification settings
- Fork 3.7k
Merged
davepagurek merged 3 commits intoprocessing:dev-2.0from
Feb 7, 2026Merged
Add transform support to bezier shapes in clip()#8481
davepagurek merged 3 commits intoprocessing:dev-2.0from
Add transform support to bezier shapes in clip()#8481
davepagurek merged 3 commits intoprocessing:dev-2.0from
Conversation
Copy link
Contributor
VANSH3104
commented
Feb 1, 2026
VANSH3104
commented
Resolves #8475 where bezier shapes (beginShape/bezierVertex) did not
respect transforms when used in clipping masks, while primitive shapes
worked correctly after PR #8236.
Changes:
- Store _clipBaseTransform in beginClip() for reference coordinate space
- Update drawShape() to apply transform matrix via addPath(path, transform)
- Maintains existing primitive shape transform handling from Fix transforms in clip() by using shape system for primitives using manual method #8236
- All shapes now consistently respect transforms in clipping masks
Screenshots of the change:
PR Checklist
-
npm run lintpasses - Inline reference is included / updated
- Unit tests are included / updated
Copy link
Contributor Author
VANSH3104
commented
Feb 1, 2026
VANSH3104 commented
Feb 1, 2026@davepagurek, could you please review this when you have a moment and suggest any necessary changes?
Copy link
pbayer8
commented
Feb 1, 2026
pbayer8 commented
Feb 1, 2026@VANSH3104 just want to thank you personally for the quick fix! My art project is relient on these bezier clip paths :)
davepagurek approved these changes Feb 7, 2026
Copy link
Contributor
davepagurek
left a comment
davepagurek
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix!
davepagurek added 2 commits
February 7, 2026 16:32
ksen0
added this to p5.js 2.x 🌱🌳
github-project-automation
bot
moved this to Completed
in p5.js 2.x 🌱🌳
ksen0
added this to the
2.3 milestone
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
