Skip to content

Fix typo in directionalLight reference (dev-2.0 branch) #7743 #7778

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

LalitNarayanYadav
Copy link
Contributor

Title

Fix incorrect reference to p5.Geometry in directionalLight() documentation

Description

This PR corrects a typo in the documentation comment for the directionalLight() function.

  • Before: direction sets the light’s direction using a p5.Geometry object.
  • After: direction sets the light’s direction using a p5.Vector object.

Since p5.Vector is the correct type used for directional light direction, this change improves the accuracy and clarity of the reference documentation in version 1.11.

@LalitNarayanYadav
Copy link
Contributor Author

fixes #7743 for version 2.0

@limzykenneth
Copy link
Member

@dhowe @davepagurek It looks to me like the Google Font is not loaded in the visual test case for some reason causing the test fail.

@davepagurek
Copy link
Contributor

I reran the tests and they passed, it seems like this one is flaky for some reason. I'm not sure how worrisome that is -- maybe there was a fluke accessing Google when the tests ran? but let's keep an eye out for this same test failure in case it's something to do with the fonts implementation.

@davepagurek davepagurek merged commit 9cd8444 into processing:dev-2.0 May 8, 2025
2 checks passed
@davepagurek
Copy link
Contributor

@all-contributors please add @LalitNarayanYadav for docs

Copy link
Contributor

@davepagurek

@LalitNarayanYadav already contributed before to doc

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.

3 participants