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

Project Manager shows (DEBUG) as title #100727

Open
timothyqiu opened this issue Dec 22, 2024 · 6 comments · May be fixed by #100745
Open

Project Manager shows (DEBUG) as title #100727

timothyqiu opened this issue Dec 22, 2024 · 6 comments · May be fixed by #100745

Comments

@timothyqiu
Copy link
Member

Tested versions

v4.4.dev7.official [46c8f8c]

System information

Godot v4.4.dev7 - Arch Linux #​1 SMP PREEMPT_DYNAMIC Mon, 09 Dec 2024 14:31:57 +0000 on X11 - X11 display driver, Multi-window, 2 monitors - OpenGL 3 (Compatibility) - NVIDIA GeForce RTX 3060 Ti (nvidia; 565.77) - 13th Gen Intel(R) Core(TM) i5-13400F (16 threads)

Issue description

The title is the correct Godot Engine - Project Manager when loading (showing the splash screen). It turns into (DEBUG) when loading is done:

Image

Steps to reproduce

Open the project manager.

Minimal reproduction project (MRP)

N/A

@timothyqiu timothyqiu added this to the 4.4 milestone Dec 22, 2024
@Giganzo
Copy link
Contributor

Giganzo commented Dec 22, 2024

Doesn't happen on v4.4.dev5.official [9e60984]

@mdelorme
Copy link
Contributor

mdelorme commented Dec 22, 2024

Bisecting seems to show the culprit is commit ff6eea5

What I think is happening is :

  1. The project manager window is given a title at creation by directly going through the DisplayServer without using the Window node here)
  2. The scene tree is initialized with everything.
  3. Post "enter tree", commit ff6eea5 sends a NOTIFICATION_TRANSLATION_CHANGED to the nodes
  4. The Window node receives the notification, and re-translates the title here. However, to do so it uses the title variable that has never been set. The title is thus empty, but since we're on a dev version (with DEBUG_ENABLED active), (Debug) is appened at the end of the empty string. Giving thus the wrong title.

I'm really not sure how to proceed to fix this, because by the time the title is first set, the Window node does not exist and so we cannot set the title variable properly.

@Mickeon
Copy link
Contributor

Mickeon commented Dec 22, 2024

Bisecting seems to show the culprit is commit ff6eea5

CC @AThousandShips

@AThousandShips
Copy link
Member

AThousandShips commented Dec 22, 2024

Then the older more extensive solution I had might have been required, this particular change can't really safely be reverted as it solves a pretty serious crash

Can't really work on a fix for that part right now, but it shouldn't be relying on that detail I think, so something else is likely wrong

CC @YeldhamDev

@AThousandShips
Copy link
Member

Some other changes must have happend to cause this particular problem as the translation update used to be in this part of the initialization process, so something else must have shifted to make this a problem, the title has to be updated correctly probably at an earlier phase

@mdelorme
Copy link
Contributor

mdelorme commented Dec 22, 2024

Ok, I think we can solve simply the problem. I'm sorry I made a mistake in my assessment: the root of the scene already exists when we give the title to the project manager initially. So we can give it a title instead of going directly to the DisplayServer. I'm testing a few more things and if that works I'll try to push a fix.

mdelorme added a commit to mdelorme/godot that referenced this issue Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Bad
Development

Successfully merging a pull request may close this issue.

6 participants