Skip to content
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

Fix examples with broken #1506

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

davepagurek
Copy link
Contributor

Fixes https://github.com/processing/p5.js-website/issues/1505

Changes:

  • For orbit control, removes the translate() call, and uses a camera to place the view at the origin
  • For raycasting, updates the distance from the camera to 800, the new default

Screenshots of the change:

@limzykenneth limzykenneth merged commit 341f5a0 into processing:main Feb 19, 2024
1 check passed
@limzykenneth
Copy link
Member

Thanks @davepagurek. Do tag me for review if there are somewhat critical fixes like this. I'll work with Q to get a new p5.js release at some point so if there are reference examples that need to be updated they can be done soon. Although at the same time do also let me know if you need me to work on and help with any of the STF stuff you have as it seems you have a lot on your plate already.

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.

Camera related examples look different after 1.9.0
2 participants