← Back to team overview

widelands-dev team mailing list archive

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