Merge lp:~hjd/widelands/metalworks-text-bug-734409 into lp:widelands
Proposed by
Hans Joachim Desserud
Status: | Merged |
---|---|
Merged at revision: | 5921 |
Proposed branch: | lp:~hjd/widelands/metalworks-text-bug-734409 |
Merge into: | lp:widelands |
Diff against target: |
21 lines (+2/-2) 1 file modified
campaigns/t03.wmf/scripting/texts.lua (+2/-2) |
To merge this branch: | bzr merge lp:~hjd/widelands/metalworks-text-bug-734409 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Nasenbaer | Approve | ||
SirVer | Approve | ||
Review via email: mp+63467@code.launchpad.net |
Description of the change
(See bug 734409 for details)
Reworked part of the campaign text to avoid confusion around axe factories and blacksmith/master blacksmith.
The change on line 336 is the one who is shown in the story dialog, while the change on line 111 is the reminder shown if the player checks current objectives.
To post a comment you must log in.
From me a disapprove - of course your text is nicer to read and I generally like the new texts, but the main problem for me is: the player reads a long sentance and stops somewhere as (s)he is quite sure (s)he already got the point - so propably (s)he misses the *very important* note, that (s)he should only upgrade the building, if the worker reaches the next level....