Updated point() examples for visible points in main branch. #8305
Updated point() examples for visible points in main branch. #8305perminder-17 merged 5 commits intoprocessing:mainfrom
Conversation
|
🎉 Thanks for opening this pull request! For guidance on contributing, check out our contributor guidelines and other resources for contributors! Thank You! |
|
Hi @perminder-17 , As suggested, I’ve created this PR for the main branch. |
|
Hey @perminder-17, I have fixed the issues. Please review and let me know if any changes are needed. |
|
|
||
| this.pointSize = 5.0; //default point size | ||
| this.curStrokeWeight = 1; | ||
| this.pointSize = this.curStrokeWeight; |
There was a problem hiding this comment.
Thanks. This change makes the behavior match the docs, but the concern is that users who rely on the old default point size in their sketches will see a visual change and will now need to add strokeWeight(5) to keep the old look.
2 participants
Resolves #8266
Changes:
There were three examples of point() in the docs which I updated and made them visible as compared to before for the main branch. (similar that has done for dev-2.0 branch)
Screenshots of the change:
before:
after:
PR Checklist
npm run lintpasses