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

Overwrite render pass to Some/None instead of just setting it to Some #824

Closed
wants to merge 1 commit into from

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Sep 24, 2024

#719 changed this to only ever set the render target to Some if the camera's is Some, but never unset it to None.

In my game I currently need to reset with set_camera_default() before setting a camera that doesn't have a render target set.

@profan
Copy link
Sponsor Contributor

profan commented Sep 29, 2024

Seems like this was fixed in 4ad23c8, can probably be closed now :)

(also sorry 👁️)

@oli-obk oli-obk closed this Sep 29, 2024
@oli-obk
Copy link
Contributor Author

oli-obk commented Sep 29, 2024

Ah great, thanks!

@oli-obk oli-obk deleted the stuck_camera branch September 29, 2024 14:12
@not-fl3
Copy link
Owner

not-fl3 commented Sep 29, 2024

Ohh shame on me, solution was there in PRs all along

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