◐ Shell
reader mode source ↗
Skip to content

Update docs for setHeading#1015

Merged
SableRaf merged 3 commits into
processing:mainfrom
babaissarkar:update_set_heading
Jul 11, 2025
Merged

Update docs for setHeading#1015
SableRaf merged 3 commits into
processing:mainfrom
babaissarkar:update_set_heading

Conversation

@babaissarkar

@babaissarkar babaissarkar commented Mar 26, 2025 •
edited
Loading

Copy link
Copy Markdown

Resolves #472

Adds JavaDoc for PVector#setHeading() and the associated keywords.txt entry, generated using the instruction at https://github.com/processing/processing-website?tab=readme-ov-file#keywordstxt. This allows the Syntax Highlighting and Open Content Reference context menu item to work.

(Note: Open Content Reference will open missing page until processing/processing-website#621 is merged and properly deployed.)

(Note 2: Also updates the documentation for PVector#heading() to be more explanatory, inspired by https://p5js.org/reference/p5.Vector/heading, so that JSON might also require regeneration. (Done) )

Adds more missing keywords to the keywords.txt file. (Generated via keywords.txt update routine)

@babaissarkar

babaissarkar commented Mar 26, 2025

Copy link
Copy Markdown
Author

While running the update routine for keywords.txt I came across some API functions that were missing from it, so I added them in 7d691d8. I've checked that the online reference pages for those functions already exist, they were just missing syntax highlight/content reference support.

@Stefterv Stefterv self-requested a review April 15, 2025 05:43
@Stefterv Stefterv added this to the 4.4.5 milestone May 12, 2025
@SableRaf SableRaf merged commit 4fc1c5d into processing:main Jul 11, 2025
@SableRaf

SableRaf commented Jul 11, 2025

Copy link
Copy Markdown
Collaborator

@all-contributors please add @babaissarkar for code and documentation

@allcontributors allcontributors Bot mentioned this pull request Jul 11, 2025
Merged
@allcontributors

allcontributors Bot commented Jul 11, 2025

Copy link
Copy Markdown
Contributor

@SableRaf

I've put up a pull request to add @babaissarkar! 🎉

@SableRaf

SableRaf commented Jul 11, 2025

Copy link
Copy Markdown
Collaborator

Thanks @babaissarkar for the great work and for your patience! Your change will go out in the next release of Processing 4.4.5.

We'd be happy to see you contribute further. Feel free to look at other open issues and/or join the Processing Community Discord. And let me know if you need any pointers 💙

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Stefterv Stefterv Stefterv approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

Processing Roadmap
Status: Completed +4 more

Milestone

4.4.5

Development

Successfully merging this pull request may close these issues.

PVector's setHeading() isn't highlighted, or in the reference.

3 participants

@babaissarkar @SableRaf @Stefterv