-
-
Notifications
You must be signed in to change notification settings - Fork 56
PVector's setHeading() isn't highlighted, or in the reference. #472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Created by: ExpandingS I've had some more time to look at it, I see that the highlighting comes from keywords.txt, which gets updated periodically. So just adding this doclet be sufficient? |
From the notice on top of
Also, is there a way to manually run the updater for @Stefterv any idea? |
@SableRaf Can you please review my PR |
Hey @SushantBansal-tech. Thanks so much for your enthusiasm! Just a quick note: this issue was already assigned, and another contributor had shown clear interest too. To keep things fair (and organized) please only work on issues you've officially been assigned. Also, as mentioned in our GSoC Guidelines:
We get notified of new PRs by GitHub and there’s no need to ask for a review right away. Thanks for your understanding and patience! 💙 |
@SableRaf @Stefterv seems like this is more an issue for |
@babaissarkar Great spelunking, could you start with creating a writeup on how this actually works? As @SableRaf and I are also a bit fuzzy on the details |
Sure, where should I write it? (Haven't got all details yet, still investigating.) |
At the bottom of the website repo there is a little bit of documentation, if you could append your finding there |
Opened a sister issue at https://github.com/processing/processing-website/issues. Since I'm already assigned to this, would be nice if I could be assigned there as well. |
After running the keywords.txt update routine, seems like there are more missing keywords than just @Stefterv @SableRaf what to do? should I just pick the entry for |
@babaissarkar Thanks for doing the research! Yes, in that case I think it makes sense to fix the overarching issue even though it goes beyond the scope of the initial bug report. |
Created by: ExpandingS
Description
The method setHeading isn't highlighted like other methods in PVector. It also isn't in the reference (But this may be an issue for processing-docs?).
Expected Behavior
setHeading appears with that same blue colour.
Current Behavior
Steps to Reproduce
Your Environment
Possible Causes / Solutions
I've tried adding annotations as there aren't currently any, like:
However it didn't seem to change anything.
Other
Added in this issue
The text was updated successfully, but these errors were encountered: