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

Make window title LabelText instead of LocalizedString #869

Merged
merged 1 commit into from
Apr 23, 2020

Conversation

cmyr
Copy link
Member

@cmyr cmyr commented Apr 23, 2020

It isn't really possible for code outside of druid to use
LocalizedString in any reasonable way at the moment, and so
this makes it much easier to set a window title, especially
a title that is generated dynamically from the data.

It isn't really possible for code outside of druid to use
LocalizedString in any reasonable way at the moment, and so
this makes it much easier to set a window title, especially
a title that is generated dynamically from the data.
cmyr added a commit to linebender/runebender that referenced this pull request Apr 23, 2020
This will now update when the glyph name changes.

This depends on linebender/druid#869
Copy link
Collaborator

@luleyleo luleyleo left a comment

Choose a reason for hiding this comment

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

The need for LocalizedString always annoyed me when I just wanted to set a static title 😅
I think this makes more sens.

@cmyr cmyr merged commit 7445345 into master Apr 23, 2020
@cmyr cmyr deleted the window-label-text branch April 23, 2020 18:13
cmyr added a commit to linebender/runebender that referenced this pull request Apr 23, 2020
This will now update when the glyph name changes.

This depends on linebender/druid#869
cmyr added a commit to linebender/runebender that referenced this pull request Apr 23, 2020
This will now update when the glyph name changes.

This depends on linebender/druid#869
cmyr added a commit to linebender/runebender that referenced this pull request Apr 23, 2020
This will now update when the glyph name changes.

This depends on linebender/druid#869
@zaynetro
Copy link
Collaborator

Thanks! This is a great change. I always needed to copy-paste setting window title from another example before.

@cmyr
Copy link
Member Author

cmyr commented Apr 24, 2020

yes, sorry for making it wait so long 😅

@cmyr cmyr mentioned this pull request Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants