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

Merged
merged 1 commit into from
Dec 30, 2024

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.

@mdelorme
Copy link
Contributor Author

I'm sorry, is that not wanted behavior ?
Should we aim for Godot Engine - Project Manager without the (DEBUG) even in debug builds ?

@timothyqiu
Copy link
Member

Should we aim for Godot Engine - Project Manager without the (DEBUG) even in debug builds ?

Yes. That's the previous behavior. Editor builds always define DEBUG_ENABLED macro.

@mdelorme mdelorme requested a review from a team as a code owner December 23, 2024 09:25
@mdelorme
Copy link
Contributor Author

@timothyqiu I proposed a fix, can you tell me if that sounds correct to you. If that's ok, I'll squash everything.

@mdelorme mdelorme force-pushed the mdelorme/issue_100727 branch from 5e360f1 to 83e7440 Compare December 23, 2024 09:27
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.

Works as expected.

@mdelorme mdelorme force-pushed the mdelorme/issue_100727 branch 2 times, most recently from e5ec66c to 51cc14a Compare December 23, 2024 18:31
@mdelorme
Copy link
Contributor Author

I have added a comment on excluding the project manager for the (DEBUG) suffix and I've included the fix to set_title so that everything is done correctly everywhere.

@mdelorme mdelorme force-pushed the mdelorme/issue_100727 branch from 51cc14a to e4e966b Compare December 23, 2024 18:44
@Repiteo Repiteo merged commit 01c6e18 into godotengine:master Dec 30, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Dec 30, 2024

Thanks!

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
4 participants