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 certain editor plugins not showing when they should #98675

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

YeldhamDev
Copy link
Member

@YeldhamDev YeldhamDev commented Oct 30, 2024

And by "certain editor plugins", I mean the Animation editor.

The issue happens when an editor plugin is activated, hidden, but then requested again by something else. When a plugin was already activated, make_visible() wasn't called on it.

Fixes #93814.

Sponsored By: 🐺 Lone Wolf Technology / 🍀 W4 Games.

@KoBeWi
Copy link
Member

KoBeWi commented Dec 7, 2024

Clicking AnimationTree will always open Animation tab.

RmVxy9ktkT.mp4

@YeldhamDev
Copy link
Member Author

@KoBeWi This behavior is due to a different issue (discussed in godotengine/godot-proposals#11009), and is reproducible without this PR.

@KoBeWi
Copy link
Member

KoBeWi commented Dec 8, 2024

Not entirely.

godot.windows.editor.dev.x86_64_ieaEL62Jka.mp4

@YeldhamDev
Copy link
Member Author

Alright, I made so that the _plugin_over_edit() call is deferred, and it will make it work like before. This is a band-aid, and the true solution would be a proper priority system, which is not in the scope of this PR.

@Repiteo Repiteo merged commit 15aa18b into godotengine:master Dec 10, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Dec 10, 2024

Thanks!

@YeldhamDev YeldhamDev deleted the dont_be_shy_plugin branch December 10, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants