widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #11965
Re: [Merge] lp:~widelands-dev/widelands/bug_1736095_improve_text_for_resources_height into lp:widelands
I found another "Set Value" in the "Random height" tool and changed it.
> What do you think?
You're asking the wrong person here, i am struggling already with german spelling rules :-D
But having a general rule is good.
I tried to run through all menus with those rules in mind, at least in the editor, and found three things:
Terrains: The tooltip for each terrain says "Name: properties" starting lowercase after the colon. Not sure if this is good practice in english.
Load/Save map: There is the string "Current Directory". It should be "Current directory"? If the "Title Rule" applies here, there are other terms which should be changed, eg. in Map Options: "Map name:" -> "Map Name:" or in Random Map: "Random number" -> "Random Number"?
Bottom menu: There we have e.g. "Main Menu" and "Tool Size" but "Reset zoom" (lowercase "z"). If we apply the rule here, it should be e.g. "Main menu"? On the other side i think the bottom menu is very importand and the Tooltips are more like a Window title. So the rule "title case" would also fit and we change "Reset zoom" to "Reset Zoom"?
--
https://code.launchpad.net/~widelands-dev/widelands/bug_1736095_improve_text_for_resources_height/+merge/335130
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug_1736095_improve_text_for_resources_height.
References