-
-
Notifications
You must be signed in to change notification settings - Fork 18.8k
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
Error message whenever unfocusing editor window while working on a saved scene. #89570
Comments
Can you test this in 4.2.2.rc2? This sounds familiar and might be fixed However this doesn't seem like something that's universal, or we'd see a lot of reports, so it's very likely specific to your project so we'd need to have an example of it happening, it doesn't happen in a new project |
I ran some tests and received some results. Summary: Here are the unabridged details for each test: Note: Both Tests 1 and 2 were not editor-breaking Test 1v4.0.stable - New Project
Test 2v4.2.1.stable - New Project
Test 3:4.2.2.rc2 - New Project (after saving scene for first time)
Along with the following error. After that, it proceeded to output this error on each unfocus:
Could be reproduced by following the steps to reproduce:
Notes:
Test 44.2.2.rc2 - Previous Project (loading the scene saved from Test 3)
Test 54.2.1.stable - New Project
On unfocus:
Test 64.2.1.stable - Previous Project (loading scene saved from Test 5)
Test 74.0.stable - New Project
Paths:
On unfocus (alongside a blank error):
Uniquely to 4.0.stable, it seems that only the first scene save on brand new projects causes this error. Upon opening up the new project after closing it and saving a new scene, we get the usual results from Test 1 rather than the editor-breaking ones. However, I did manage to reproduce the results from Test 7 on a brand new project on the same version. |
Please delete your editor settings and similar and try, this isn't a bug in the editor, and unless this is an issue that can be solved by a change in the engine, and not something that's broken in your drivers, or configuration, there's nothing we can do really, so please see if clearing your data helps |
Update: I believe this is caused by |
Can confirm, looked this thread up because I had the exact same problem. Disabling |
Godot v4.3.dev (5d08c26) - macOS 14.4.1 - Vulkan (Mobile) - integrated Apple M1 Pro - Apple M1 Pro (10 Threads) |
@AThousandShips Save on Focus is handled via Node::propagate_notification, which increases data.blocked before propagation. Since Save Dialog is displayed on the root Window (due to absence of any focused windows), it always fails. |
This isn't my area of expertise so no use pinging me with details, but hopefully someone can use that information |
Duplicate of #73765 |
Thanks, consolidating in #73765. |
Tested versions
System information
Godot v4.2.1.stable - Windows 10.0.22631 - GLES3 (Compatibility) - Intel(R) Iris(R) Xe Graphics (Intel Corporation; 31.0.101.1999) - 12th Gen Intel(R) Core(TM) i7-1260P (16 Threads)
Issue description
This error is written to Output for each time I click out of the editor on any project when working on a saved scene:
I don't think I've ever seen that path or its number change (
scene/main/window.cpp:1658
).This happens in new projects too. To my knowledge, it doesn't do anything to really cause any real problems. I do have some other issues, but I think those are separate matters (mainly, nested custom resources breaking and returning null on use but still showing up in the stack trace and inspector).
*I would also like to mention that this is my first time using GitHub to contribute anything. Feel free to let me know if I'm doing anything wrong
Steps to reproduce
The error occurs whenever the editor loses focus (ex. when I click outside of the window). This happens in both my main project that I've been working on, but also a brand new project.
However, it only happens when inside a saved scene (I tested a plain Node2D scene and a Node3D scene). If I am in an unsaved 'new tab' labelled
[empty]
or another unsaved new scene, then the editor does not give the error.Minimal reproduction project (MRP)
N/A
The text was updated successfully, but these errors were encountered: