Minor string issues

Bug #1300724 reported by wl-zocker
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Unassigned

Bug Description

(I hope it is ok to put that in one bug report because it is simply adding an "e" or a space at some places. If there is a possibility to fix that without having to download the whole source code, let me now. Then I will not bother you with such issues any more.)
- There are two strings that contain the word "axs", but the correct plural is "axes": https://translations.launchpad.net/widelands/trunk/+pots/tribe-barbarians/de/397/+translate and https://translations.launchpad.net/widelands/trunk/+pots/scenario-t02.wmf/de/49/+translate
- In this string, a space character is missing between "Fremil." and "It": https://translations.launchpad.net/widelands/trunk/+pots/scenario-emp02.wmf/de/80/+translate

I think these strings appear in the game when the language is English. Maybe I stumble across some more.

- For the Barbarians and Atlanteans, the building is called "Weaving-Mill", but the Empire has a "Weaving Mill" (without hyphen). I do not know which one is correct American English. There is one exception in the Atlantean campaign, where the building is referred to as "weaving mill": https://translations.launchpad.net/widelands/trunk/+pots/scenario-atl01.wmf/de/26/+translate
- There must not be a comma between "I think" and the rest. Furthermore, the "and" at the end makes no sense: https://translations.launchpad.net/widelands/trunk/+pots/scenario-atl01.wmf/de/104/+translate
- "a special building where the side effects" instead of "were": https://translations.launchpad.net/widelands/trunk/+pots/scenario-atl01.wmf/de/89/+translate
- The are two strings that could be put together for easier translation: https://translations.launchpad.net/widelands/trunk/+pots/scenario-tutorial01.wmf/de/142/+translate and https://translations.launchpad.net/widelands/trunk/+pots/scenario-tutorial01.wmf/de/143/+translate (they have an image in between, which should follow the whole string).
- Same goes for the strings 189 and 190 https://translations.launchpad.net/widelands/trunk/+pots/scenario-tutorial01.wmf/de/+translate?memo=187&start=187. I think the first part (188) cannot be put to them easily. I attach a screenshot for these two for better representation.

Related branches

wl-zocker (wl-zocker)
description: updated
wl-zocker (wl-zocker)
description: updated
SirVer (sirver)
Changed in widelands:
assignee: nobody → GunChleoc (gunchleoc)
Revision history for this message
wl-zocker (wl-zocker) wrote :
description: updated
Revision history for this message
GunChleoc (gunchleoc) wrote :

Thanks for catching these, I will fix them.

The last two will stay as they are. combining them would mean adding line break code to the string, which is bad. Since the strings now appear in order in the po file, context should be no problem.

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

>I hope it is ok to put that in one bug report because it is simply adding an "e" or a space at some places.

Absolutely. :)

>If there is a possibility to fix that without having to download the whole source code, let me now.

I believe there are some tricks to make the checkout smaller if disk space is the problem, but you will unfortunately need the source code.

A minor note (because this is rather counter-intuitive): In some cases it might be better to include the strings themselves than links/string numbers. Whenever new translation templates are generated it gathers any translatable strings, which means the amount of strings may have changed and/or the order the strings appear in. This means that any old links to string 42 might suddenly point to something completely different. This isn't usually a problem since tiny string changes are resolved rather quickly, but just so you know. :)

Changed in widelands:
status: New → Triaged
importance: Undecided → Low
SirVer (sirver)
Changed in widelands:
milestone: none → build19-rc1
GunChleoc (gunchleoc)
Changed in widelands:
status: Triaged → In Progress
SirVer (sirver)
Changed in widelands:
status: In Progress → Fix Committed
GunChleoc (gunchleoc)
Changed in widelands:
assignee: GunChleoc (gunchleoc) → nobody
GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

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.