Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Incomplete Translation for artifacts in the Intelligence Display #4843

Closed
wzdev-ci opened this issue Nov 18, 2018 · 15 comments
Closed

Incomplete Translation for artifacts in the Intelligence Display #4843

wzdev-ci opened this issue Nov 18, 2018 · 15 comments

Comments

@wzdev-ci
Copy link
Contributor

resolution_fixed type_bug | by alfred007


When you click on a researched artifact in the Intelligence Display you get a description of the artifact. The title at the top and at the bottom is still in English, the description in the middle is translated. I add a picture to display the bug for the German translation, but this is the case in all languages.


Issue migrated from trac:4843 at 2022-04-16 13:33:00 -0700

@wzdev-ci
Copy link
Contributor Author

alfred007 uploaded file Incomplete Translation.jpg (245.2 KiB)

Incomplete Translation.jpg

@wzdev-ci
Copy link
Contributor Author

Forgon changed status from new to assigned

@wzdev-ci
Copy link
Contributor Author

Forgon changed owner from `` to Forgon

@wzdev-ci
Copy link
Contributor Author

Forgon changed blocking which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

Forgon changed blockedby which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

Forgon commented


Thank you, I will fix this.
There are several similar problems.
For example, challenge and AI descriptions are largely untranslated.

@wzdev-ci
Copy link
Contributor Author

alfred007 commented


I think I have to clarify the situation. When you pick up the artifact in the game, the name of the artifact is translated. But in the Intelligence Display, the same name is not translated. So it's not necessary to translate it again but only to put a link into the .po file.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Mar 9, 2019

Berserk Cyborg commented


Couldn't we encapsulate the research names in intelmap.cpp with _()? Lines 320 and 456. Can' test this though.

psResearch->name.toUtf8().c_str() -> _(psResearch->name.toUtf8().c_str())

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Mar 17, 2019

Forgon commented


Replying to Warzone2100/old-trac-import#4843 (comment:3):

Couldn't we encapsulate the research names in intelmap.cpp with _()? Lines 320 and 456. Can' test this though.

psResearch->name.toUtf8().c_str() -> _(psResearch->name.toUtf8().c_str())
Yes, this works.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Mar 17, 2019

Forgon commented


Replying to Warzone2100/old-trac-import#4843 (comment:2):

I think I have to clarify the situation. When you pick up the artifact in the game, the name of the artifact is translated. But in the Intelligence Display, the same name is not translated. So it's not necessary to translate it again but only to put a link into the .po file.
This is not how gettext works.
In any case, the problem is fixed now.

@wzdev-ci
Copy link
Contributor Author

Forgon uploaded file intelligence_screen_translation_new.png (407.4 KiB)

translated title and button tooltip for research message in intelligence screen
intelligence_screen_translation_new.png

@wzdev-ci
Copy link
Contributor Author

Forgon uploaded file research_message_translation.patch (1.2 KiB)

patch file based on revision 356710b5c7efed84054b3b70003a6539dfb46265

@wzdev-ci
Copy link
Contributor Author

Forgon2100 <forgon2100@...> changed status from assigned to closed

@wzdev-ci
Copy link
Contributor Author

Forgon2100 <forgon2100@...> changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

Forgon2100 <forgon2100@...> commented


In Warzone2100/warzone2100@f1b170d:

#CommitTicketReference repository="" revision="f1b170db3012f4fcd99335cd814180ea8d37e0d6"
Fix translation of research messages

* show translations for titles and button tooltips of research messages
  in the intelligence screen

Fixes #4843

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

No branches or pull requests

1 participant