godotengine
Can't open Animation tab with if a shader resource is expanded in the inspector, reverts to Shader Editor again
Godot version
4.0beta10
System information
macOS 13.0.1
Issue description
If you expand a shader resource in the inspector panel this automatically opens the Shader Editor tab in the bottom panel. However, there seems to be a bug with the Animation tab that somehow triggers this again when the animation tab is opened (maybe it refreshes the inspector?), resulting in the Shader Editor immediately being opened again. So, if you have a shader resource expended in the inspector you can't open the Animation tab to edit animations. If you just collapse the shader resource again it works as expected.
https://user-images.githubusercontent.com/17763524/209969815-adb9cd73-1cd7-4756-9af2-9042ccec9442.mov
I tried this in 4.0 beta 8 as well, and interestingly it had the same problem but also another bug/hack that caused the expanded shader resource to be collapsed when it switched back to the Shader Editor. So if you clicked the Animation tab a second time you could now switch to it.
Steps to reproduce
Create new scene (2d or 3d) Add an AnimationPlayer to the scene so there's something in the Animation tab Open the animation tab with the Animation player selected Create something (2d or 3d) that has a material Assign a new shader material Create a new shader for the material Expand the material and shader resources in the inspector Notice that this opens the shader editor in the bottom Now try to open the animation tab, notice it will switch back to the shader editor tab immediately
Minimal reproduction project
I can confirm this on 4.0.beta10 while working on https://github.com/godotengine/godot-demo-projects/pull/809.
This bug, and probably https://github.com/godotengine/godot/issues/70588 then too has reappeared again in 4.0b15 : If you have a Shader resource expanded in the inspector you can't switch to the Animation tab as it will keep the Shader Editor tab open.
Caused by #70909 It's the same issue as the one fixed by the reverted PR, i.e. pressing Animation will reload inspector and reopen all properties, triggering unwanted editors.
Now that #71770 has been merged, it may be worth re-applying the two commits that were Reverted in #70909, if the issue in #70914 does not occur again.