missing whitespace require translating the almost same map description multiple times

Bug #899161 reported by Borim
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Hans Joachim Desserud

Bug Description

The map description from Four Castles contain the string:
"Suggested Teams: (blue,red) vs (yellow,green)"

the map Swamp Monks contains a similar string:
"Suggested Teams: (blue, red) vs (yellow, green)" (with whitespaces)

due to this small difference you have to translate the same sentence multiple times. Also other maps like Four Mountains have comparable strings...

For uniformity reasons and to reduce the work for translation, a common sentence structure should be used. Either with or without whitespace.

Related branches

Borim (borim)
summary: - typo in the map Four Castles require translating the same text multiple
- times
+ missing whitespace require translating the almost same map description
+ multiple times
Revision history for this message
SirVer (sirver) wrote :

I prefer with whitespaces and agree.

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

I can fix this one. :)

Changed in widelands:
status: New → Triaged
importance: Undecided → Low
milestone: none → build17-rc1
assignee: nobody → Hans Joachim Desserud (hjd)
tags: added: gamedata i18n internationalization
Changed in widelands:
status: Triaged → In Progress
Tino (tino79)
Changed in widelands:
status: In Progress → Fix Committed
Revision history for this message
Angelo Locritani (alocritani) wrote :

even if the fix has been committed to trunk, in the online translations section both strings (with and without whitespace are present. (ie: https://translations.launchpad.net/widelands/trunk/+pots/maps/it/+translate?show=untranslated)

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

As I mentioned in the review, I believe utils/buildcat.py needs to be run in order to generate new translation templates to replace the existing ones.

Revision history for this message
SirVer (sirver) wrote :

Done in r6168. launchpad will need ~24hours to pick up the changes though.

Revision history for this message
SirVer (sirver) wrote :

Released in build17-rc1.

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.