Topic: Editor help
kaputtnik
Joined: 2013-02-18, 19:48
Posts: 2439 OS: Archlinux Version: current master  One Elder of Players Location: Germany |
Posted at: 2015-12-12, 15:40
Maybe i couldn't follow your discussion, but from my point of view the "(Old)" statement should be truncated from the tree name. It has no useful meaning if it stays here and may confuse more than it helps. Regarding the terrain affinity values: Those are also more confusing if they are in the editor help. In my opinion they belong to the info tool (shortcut "i"). The editor help should be a help for beginners, not for advanced players. Edit: Numbering of visual equal terrains should be come back. Edited: 2015-12-12, 15:57
Fight simulator for Widelands: 
Top
Quote
|
|
wl-zocker
Joined: 2011-12-30, 16:37
Posts: 495  Tribe Member Location: Germany |
Posted at: 2015-12-12, 16:58
When you can choose in the editor which tree to place, the age of the tree should stay in the name. For some trees, it might be hard to see if they are old or only mature. For consistency, all trees should have such an attribute. This is why I wrote:
The problem is that we do never have the tree name "Alder" without attributes; however, this is what should be used in the new help window. A simple truncating of the string is not possible because of the translations. I see in general two ways to achieve this:
We should discuss who is the target group for this information window, and which information are appropriate. Furthermore, the information which terrains are good for trees (in general, no specification for different tree species needed) could be interesting during playing, too; whereas the other tab (which terrain is good for alders) is not important. "Only few people know how much one has to know in order to know how little one knows." - Werner Heisenberg
Top
Quote
|
GunChleoc
Topic Opener
Joined: 2013-10-07, 14:56
Posts: 3324 One Elder of Players Location: RenderedRect |
Posted at: 2015-12-13, 08:58
This won't work, because the engine can't do fancy stuff on strings for the descname. If we add a %s here, we will have to add it to all immovables. What we might do is add another variable _"%s (Old)" etc. with %s being the tree name. It will create problems with my translations though, because I have a special word for saplings - other languages might have the same problem. My 4 translations for "Alder" are:
I could work around this, but the translation would be less elegant.
We could add that to the Old trees only, no problem. Busy indexing nil values
Top
Quote
|
|
GunChleoc
Topic Opener
Joined: 2013-10-07, 14:56
Posts: 3324 One Elder of Players Location: RenderedRect |
Posted at: 2015-12-13, 12:52
Final screenshots for this year. Please feel free to play with this branch while I'm gone, e.g. we still need to write tests and to sort the terrains/trees in Lua by percentage.
Busy indexing nil values
Top
Quote
|
king_of_nowhere
Joined: 2014-09-15, 17:35
Posts: 1668 One Elder of Players |
Posted at: 2015-12-13, 14:25
Nice one. But, if I must be a perfectionist - and keeping in mind that perfectionism is the enemy of good, and that it's better to make some small improvement than to make no improvemment because you want to do it even better - it would be even better if the terrains under a trees, and the trees under a terrain, would be ordered according to their affinity. I mean, it should display like ashes trees that may grow
Top
Quote
|
|
GunChleoc
Topic Opener
Joined: 2013-10-07, 14:56
Posts: 3324 One Elder of Players Location: RenderedRect |
Posted at: 2015-12-13, 16:14
.... which is exactly what I mean by " and to sort the terrains/trees in Lua by percentage." Busy indexing nil values
Top
Quote
|
|
wl-zocker
Joined: 2011-12-30, 16:37
Posts: 495 Tribe Member Location: Germany |
Posted at: 2015-12-20, 10:52
… and which I have just added. "Only few people know how much one has to know in order to know how little one knows." - Werner Heisenberg
Top
Quote
|
|
kaputtnik
Joined: 2013-02-18, 19:48
Posts: 2439 OS: Archlinux Version: current master One Elder of Players Location: Germany |
Posted at: 2015-12-20, 12:38
Great, wl_zocker We must consider to omit terrains where tree have a minimal chance to grow. As a strange example here are dead terrains, f.e.:
Is there a need to have those terrains in the list? Same goes for all water terrains:
Yes, i know each terrain has influence about the possibility to grow of neighboring terrains. But in my opinion the help couldn't show all information. I would suggest to show only those terrains where trees have a probability to grow about >50%. All others are of no interest related to trees i think. Fight simulator for Widelands:
Top
Quote
|
|
wl-zocker
Joined: 2011-12-30, 16:37
Posts: 495 Tribe Member Location: Germany |
Posted at: 2015-12-20, 14:33
I have no problem with the dead terrains saying that no trees will grow. They're terrains in the game, and we should list them. What I found strange are the water terrains where trees can grow. This could make players think that trees sprout in the middle of the ocean. Maybe we could solve this by simply increasing the threshold from 1% to 5%? If not all terrains are shown in the left list, this would have to be done in C++, I think. "Only few people know how much one has to know in order to know how little one knows." - Werner Heisenberg
Top
Quote
|
|
kaputtnik
Joined: 2013-02-18, 19:48
Posts: 2439 OS: Archlinux Version: current master One Elder of Players Location: Germany |
Posted at: 2015-12-20, 15:56
Yes, i meant the left list when terrain tab is open. I think if this should be a help for map designers, its better to have less terrains in the list by omitting terrains where trees do not grow well. For dead terrains it should already be clear by the skull icon in terrain menu. And for water or ice floes (probability to grow for spruce: 1,9%) it should be clear also. And: It is not possible to place trees on water, ice floes, swamp or dead terrains so why should those terrains be in the list? Edited: 2015-12-20, 15:58
Fight simulator for Widelands:
Top
Quote
|
