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

Reimporting a resource makes godot open the script editor #90944

Open
passivestar opened this issue Apr 20, 2024 · 1 comment · May be fixed by #97009
Open

Reimporting a resource makes godot open the script editor #90944

passivestar opened this issue Apr 20, 2024 · 1 comment · May be fixed by #97009

Comments

@passivestar
Copy link
Contributor

passivestar commented Apr 20, 2024

Tested versions

v4.3.dev.custom_build [4a01602]

System information

Godot v4.3.dev (4a01602) - macOS 14.4.0 - Vulkan (Forward+) - integrated Apple M1 Max - Apple M1 Max (10 Threads)

Issue description

Reimporting an asset opens the script editor. If an external editor is used it opens the external editor (brings it on top)

This is distracting as it is by itself (if I have them opened side by side and want to quickly see the changes in editor I need to switch back to 3D every time I change something), and it's also a big problem when using an external editor and trying to hot-reload assets in a running game, because vscode keeps maximizing and needs to be constantly minimized

Screen.Recording.mp4

Steps to reproduce

Open the MRP, the scene inside of it and the script, and try re-exporting the cube

Minimal reproduction project (MRP)

MRP_reimport_script_editor.zip

@KoBeWi
Copy link
Member

KoBeWi commented Apr 20, 2024

Caused by this line:

_edit_current();

Only happens if you were editing a script just now and an instance in the current scene is reimported. The currently "edited" object is edited again.
It's similar to #90654. You can click a Node3D to "edit" it and prevent the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants