Let tree descriptions use the same size scheme as buildings
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
widelands |
Fix Released
|
Low
|
Unassigned |
Bug Description
The different worlds each have a conf-file which lists the different immovable types which may be placed. Among these are the trees, for instance:
tree4_m=_Maple (medium)
tree4_s=_Maple (small)
tree4_t=_Maple (tiny)
I've always found these sizes a bit odd since medium is usually the middle value and there is no large/big/huge/etc trees. Is there a reason why this scheme was chosen? I wonder if it would be better to reuse the small/medium/large scheme which is already in place for buildings and would be instantly recognizable.
I have noticed this a long time ago, but I haven't filed it before now because it is a minor issue and it would require to redo lots of translations for trees in all languages. However, we have recently done some major changes to translations which basically require to re-add a lot of translations, so this might be an opportunity to change this. ;)
Related branches
- Widelands Developers: Pending requested
-
Diff: 316 lines (+145/-128)4 files modifiedworlds/blackland/conf (+36/-32)
worlds/desert/conf (+37/-32)
worlds/greenland/conf (+36/-32)
worlds/winterland/conf (+36/-32)
tags: | added: internationalization |
Changed in widelands: | |
assignee: | GunChleoc (gunchleoc) → nobody |
Changed in widelands: | |
status: | Fix Committed → Fix Released |
well, using the same description would be confusing as well. From an engine point of view, all trees are either of immovable size none (i.e. can be destroyed by placing a flag) or small (i.e. the size of a flag or a small building). The tree descriptions could be changed to young, older, old or so, but I think it is probably not worth the effort.
+1 for won't fix. But I think gunchleoc should speak on this as it is a language issue.
@gun: feel free to unassign after you said your opinion. I realized I did not mention this before, but getting a bug assigned is like pinging here. It does not mean you need to fix it - if you do not plan it, unassign yourself :)