Skip to content

Comments

Updating docs for new text functions#7692

Merged
davepagurek merged 3 commits intoprocessing:dev-2.0from
perminder-17:patch-8
Apr 6, 2025
Merged

Updating docs for new text functions#7692
davepagurek merged 3 commits intoprocessing:dev-2.0from
perminder-17:patch-8

Conversation

@perminder-17
Copy link
Collaborator

No description provided.

* // Simplified to just draw line to endpoint
* const x = cmd[3];
* const y = cmd[4];
* vertex(x, y);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you think it's worth showing how to draw the actual curves for Q and C? i.e.:

bezierOrder(2)
bezierVertex(cx, cy)
bezierVertex(x, y)

@perminder-17
Copy link
Collaborator Author

Hi @davepagurek, when I was working for Case Q, I noticed it's actually not hitting that case. I tried console.log('something') in case Q and noticed it's actully not printing that, so I deleted the case Q and applied bezier curve at case C.

Before bezier curve
Screenshot from 2025-04-06 18-30-43

After bezier curve
Screenshot from 2025-04-06 18-30-32

Do you have any hints for case Q? or is this good to merge?

@davepagurek davepagurek merged commit 34cd557 into processing:dev-2.0 Apr 6, 2025
2 checks passed
@perminder-17 perminder-17 deleted the patch-8 branch April 6, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants