Skip to content

PVector's setHeading() highlighted - #1007

Closed
SushantBansal-tech wants to merge 2 commits into
processing:mainfrom
SushantBansal-tech:highlighted_setheading
Closed

SushantBansal-tech wants to merge 2 commits into
processing:mainfrom
SushantBansal-tech:highlighted_setheading

Conversation

@SushantBansal-tech

Copy link
Copy Markdown

This PR fixes the issue where PVector.setHeading() was not being highlighted in the Processing editor.

Changes made:
Added setHeading_ to keywords.txt
Ensured it is categorized correctly for syntax highlighting

Issue reference:
Fixes #472 )

Testing:
Verified that setHeading() now appears highlighted in the editor

No breaking changes introduced

@SushantBansal-tech SushantBansal-tech left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This line I have written for debugging

@Stefterv

Copy link
Copy Markdown
Member

Hi @SushantBansal-tech thank you for your effort, this PR contains a lot of changes not related to the actual fix, please clean those up.

@SableRaf

Copy link
Copy Markdown
Collaborator

Actually, since you haven't been assigned the related issue, I'm going to close this PR for now. We're following the same contribution guidelines as p5.js — please wait for approval and assignment before starting work on a PR. Thank you!

@SableRaf SableRaf closed this Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

4 participants