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

Assign the project manager title to the node instead of DisplayServer #100745

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdelorme
Copy link
Contributor

Fixes #100727

Project Manager title is directly given to the DisplayServer at initialization.

With the changes of ff6eea5 auto-translation triggers the renaming of the window and leading to the bug since the title attribute of the scene root was not defined.

This fix gives the title to the root of the SceneTree which solves the problem.

@AThousandShips AThousandShips changed the title Fix #100727 by giving the project manager title to the node instead of DisplayServer Assign the project manager title to the node instead of DisplayServer Dec 22, 2024
@AThousandShips AThousandShips added this to the 4.4 milestone Dec 22, 2024
Copy link
Member

@timothyqiu timothyqiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The title is now Godot Engine - Project Manager (DEBUG).

The main window's title always has this (DEBUG) suffix for debug builds. The same goes for official builds too since debugging features are enabled when compiling the editor.

I think that's why the title was set on DisplayServer instead of Window.

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

Successfully merging this pull request may close these issues.

Project Manager shows (DEBUG) as title
3 participants