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

Set window title during startup #3841

Merged
merged 1 commit into from
May 19, 2023

Conversation

HebaruSan
Copy link
Member

Problem

After #3829, the GUI window title is blank at startup:

image

It's set later, once the instance is loaded.

Cause

The window title depends on the name, version, and path of the game instance, so it's currently not set while the instance is loading.

Changes

Now we set an initial window title in GUI.Main.OnLoad containing the pieces that are not dependent on having the instance loaded:

image

The full title is still set later when the instance is loaded.

@HebaruSan HebaruSan added Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI labels May 18, 2023
@HebaruSan HebaruSan requested a review from techman83 May 18, 2023 13:47
Copy link
Member

@techman83 techman83 left a comment

Choose a reason for hiding this comment

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

Nice, a good and simple piece of polish. I like it!

@HebaruSan HebaruSan merged commit 351fa35 into KSP-CKAN:master May 19, 2023
@HebaruSan HebaruSan deleted the fix/set-title-onload branch May 19, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants