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 polygon node losing its UV toolbar #96466

Merged

Conversation

object71
Copy link
Contributor

@object71 object71 commented Sep 2, 2024

Fixes #96238

Found out that the toolbar remains hidden since it is being hidden on the node_removed event.

@object71 object71 force-pushed the fix-polygon-editor-losing-toolbar-96238 branch from da0683b to dabeaa6 Compare September 2, 2024 12:39
@AThousandShips AThousandShips changed the title Fix polygon node losing its UV toolbar by adding a call to the show Fix polygon node losing its UV toolbar Sep 2, 2024
Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find it weird that only this editor is affected. Also the fix isn't really correct, because make_visible() should be called automatically together with edit().

Then again, seems like every editor handles this differently, so we should standardize it somehow. For now it's probably fine like this.

@akien-mga akien-mga added the cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release label Sep 13, 2024
@akien-mga akien-mga merged commit 9a5d0be into godotengine:master Sep 13, 2024
19 checks passed
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 4.3.1.

@akien-mga akien-mga removed the cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Polygon2D node loses its UV toolbar when added as a child and/or removed as a child from another node.
4 participants