Fix usage of perspective() with no args by davepagurek · Pull Request #8564 · processing/p5.js

Fix usage of perspective() with no args by davepagurek · Pull Request #8564 · processing/p5.js

Skip to content

Merged

davepagurek merged 1 commit intodev-2.0from

fix/perspective

Feb 24, 2026

Merged

Fix usage of perspective() with no args#8564
davepagurek merged 1 commit intodev-2.0from
fix/perspective

Conversation

Copy link

Contributor

davepagurek commented
Feb 24, 2026

Resolves #8563

Changes:

Makes sure all camera properties are cloned when we call .clone(). Basically, perspective() clones the existing camera before applying changes to it, but since not all camera property defaults were copied over, it was using undefined in math and putting NaNs in its matrices.

Screenshots of the change:

Live: https://editor.p5js.org/davepagurek/sketches/CVVyKhUnH

PR Checklist

npm run lint passes
Inline reference is included / updated
Unit tests are included / updated

Fix usage of perspective() with no args

davepagurek merged commit 2ef7704 into dev-2.0

5 checks passed

davepagurek deleted the fix/perspective branch

February 24, 2026 19:38

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

Labels

None yet

1 participant