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 2D/3D viewport context switching issues when script editor is floating #80647

Conversation

jsjtxietian
Copy link
Contributor

@jsjtxietian jsjtxietian commented Aug 15, 2023

This pr solves issue #80588.
I added a simple function to tell whether the script editor is floating or not, and when script editor is floating, skip_main_plugin should be false so the plugin will draw itself's gizmos.

Bugsquad edit:

@jsjtxietian jsjtxietian requested a review from a team as a code owner August 15, 2023 04:46
@akien-mga akien-mga changed the title fix CollisionShape3D Shape Size handle will missing when Script editor floating Fix CollisionShape3D Shape Size handle missing when Script editor floating Aug 15, 2023
@akien-mga akien-mga added this to the 4.2 milestone Aug 15, 2023
@akien-mga
Copy link
Member

This is a duplicate of #80307 which has the exact same changes. I slightly prefer the names chosen here so let's merge this one.

@akien-mga akien-mga added topic:gui cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Aug 17, 2023
@akien-mga akien-mga changed the title Fix CollisionShape3D Shape Size handle missing when Script editor floating Fix 2D/3D viewport context switching issues when script editor is floating Aug 17, 2023
@akien-mga akien-mga merged commit c7356c7 into godotengine:master Aug 17, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

@jsjtxietian
Copy link
Contributor Author

Thank you all guys, I'm looking forward to contribute more, I'll be always happy to help the community !

@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.3.

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants