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
chore: add display name to chest prefab #5
Conversation
515c865
to
8c08567
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can make it translatable 🙃
@@ -1,4 +1,7 @@ | |||
{ | |||
"DisplayName": { | |||
"name": "Chest" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we make this translatable?
@@ -1,4 +1,7 @@ | |||
{ | |||
"DisplayName": { | |||
"name": "${coreadvancedassets:game#furnishings-chest}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One drawback of this approach is that if a screen just renders this value as a text without an explicit call to TranslationSystem#translate(...)
this unreadable string will show up...
I don't understand (yet) why using this reference string in a .ui
file directly is automatically translated, but we return this string in a programmatic binding it is not. 🤔
Minor adjustment to add a display name for chests.
Especially the feature added in Terasology/Inventory#45 will benefit of this.