← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/frisians into lp:widelands

 

The claypit/aquafarm relation is exactly the other way around – the clay pit is basic, the aqua farm should be built only later in the game. The point here is that building an aqua farm makes no sense at all if there is no (working) clay pit nearby…

@GunChleoc: Thanks for the proofreading, you found quite a lot :)
· Tavern: I think the tooltip "Skipped preparing a ration because site has fruit or bread and site has smoked meat or smoked fish" will sound weird; I´d prefer "preparing only one ration" here, so it is clear that the tavern only skipped this working program because it can produce two rations at once
· Woodcutter´s house: A copy-paste error, should be set to no_performance_text_yet
· Basic Sword: Rename to "Short Sword"?
  Helptext suggestion: "This is the basic weapon of the Frisian soldiers. Together with a fur garment, it makes up the equipment of young soldiers. Short swords are produced by the small armor smithy."
· Needles helptexts: Are you sure cloth shouldn´t be mentioned as a ware made by a seamstress?
· Reindeer helptext: That´s a bit misleading, as you can´t produce fur from existing reindeer. The reindeer farm either recruits a new reindeer, or produces fur (and meat) without touching the reindeer population.
· Recycling centre helptext: It might be more accurate to say "recycles old weapon and armor parts that have been discarded by training sites", as the amount of scrap metal produced is equal to the equipment the soldier was using until he received the latest upgrade
· Water helptext: You introduced a new typo there ;)
· Buildings Seamstress and Master Seamstress: Perhaps "Sewing Room" for the basic form and "Tailor´s Shop" for the enhanced version?
-- 
https://code.launchpad.net/~widelands-dev/widelands/frisians/+merge/336910
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/frisians.


Follow ups

References