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

Physics colliders to show correctly on both sides of sprites/circles. #559

Closed
wants to merge 1 commit into from
Closed

Conversation

VagueLobster
Copy link
Contributor

@VagueLobster VagueLobster commented Jun 25, 2022

I seriously thought this code was from @TheCherno, but was apparently wrong...
This code is apparently from #517 which i didn't even know existed (or at the very least couldn't remember existed!)
I have closed this, because i don't want to take credit from the original coder 🙂 Though the main difference between the PRs are that my code also uses the z projection visualization for 2D Boxes, which #517 doesn't!

Impact Issue/PR
Issues this solves None
Other PRs this solves None

Before:
01

And after:
02

@VagueLobster VagueLobster changed the title Physics colliders now show on both sides. Physics colliders to show on both sides of sprites. Jun 25, 2022
@VagueLobster VagueLobster changed the title Physics colliders to show on both sides of sprites. Physics colliders to show on both sides of sprites/circles. Jun 25, 2022
@VagueLobster VagueLobster changed the title Physics colliders to show on both sides of sprites/circles. Physics colliders to show correctly on both sides of sprites/circles. Jun 26, 2022
@erikrl2
Copy link

erikrl2 commented Jun 27, 2022

I'm sorry, but this is a duplicate of #517.

@VagueLobster
Copy link
Contributor Author

VagueLobster commented Jun 27, 2022

I'm sorry, but this is a duplicate of #517.

Yeah in general it is the same code, which i didn't notice was up here on github, and sorry for that. I will close this now, to not cause any trouble with anybody 🙂

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.

2 participants