Merge lp:~widelands-dev/widelands/string-fixes into lp:widelands

Proposed by GunChleoc
Status: Merged
Merged at revision: 7949
Proposed branch: lp:~widelands-dev/widelands/string-fixes
Merge into: lp:widelands
Diff against target: 116 lines (+15/-32)
4 files modified
src/editor/ui_menus/tool_change_height_options_menu.cc (+2/-10)
src/editor/ui_menus/tool_change_resources_options_menu.cc (+5/-10)
src/editor/ui_menus/tool_menu.cc (+1/-1)
src/editor/ui_menus/tool_noise_height_options_menu.cc (+7/-11)
To merge this branch: bzr merge lp:~widelands-dev/widelands/string-fixes
Reviewer Review Type Date Requested Status
Miroslav Remák Approve
Review via email: mp+291335@code.launchpad.net

Commit message

Fixed editor tooltips and renamed "Noise height" to "Random height".

Description of the change

The tooltips were creating issues with Slavic languages.

I also renamed "Noise height" to "Random height" - I think the name was too technical.

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 974. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/121219787.
Appveyor build 807. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_string_fixes-807.

Revision history for this message
Miroslav Remák (miroslavr256) wrote :

LGTM.

review: Approve
Revision history for this message
GunChleoc (gunchleoc) wrote :

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/editor/ui_menus/tool_change_height_options_menu.cc'
2--- src/editor/ui_menus/tool_change_height_options_menu.cc 2016-04-06 17:27:10 +0000
3+++ src/editor/ui_menus/tool_change_height_options_menu.cc 2016-04-08 07:36:16 +0000
4@@ -22,8 +22,6 @@
5 #include <cstdio>
6 #include <string>
7
8-#include <boost/format.hpp>
9-
10 #include "base/i18n.h"
11 #include "editor/editorinteractive.h"
12 #include "editor/tools/increase_height_tool.h"
13@@ -52,16 +50,10 @@
14 {
15 change_by_.set_tooltip(
16 /** TRANSLATORS: Editor change height access keys. **/
17- (boost::format(_("Use %s to increase, %s to decrease"))
18- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
19- % _("Click")
20- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
21- % _("Shift + Click")).str());
22+ _("Click on the map to increase, Shift + Click on the map to decrease terrain height"));
23 set_to_.set_tooltip(
24 /** TRANSLATORS: Editor set height access key. **/
25- (boost::format(_("Use %s to set to this value"))
26- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
27- % _("Ctrl + Click")).str());
28+ _("Ctrl + Click on the map to set terrain height"));
29
30 change_by_.changed.connect
31 (boost::bind
32
33=== modified file 'src/editor/ui_menus/tool_change_resources_options_menu.cc'
34--- src/editor/ui_menus/tool_change_resources_options_menu.cc 2016-04-06 17:27:10 +0000
35+++ src/editor/ui_menus/tool_change_resources_options_menu.cc 2016-04-08 07:36:16 +0000
36@@ -68,17 +68,12 @@
37 {
38 // Configure spin boxes
39 change_by_.set_tooltip(
40- /** TRANSLATORS: Editor change resources access keys. **/
41- (boost::format(_("Use %s to increase, %s to decrease"))
42- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
43- % _("Click")
44- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
45- % _("Shift + Click")).str());
46+ /** TRANSLATORS: Editor change rseources access keys. **/
47+ _("Click on the map to increase, "
48+ "Shift + Click on the map to decrease the amount of the selected resource"));
49 set_to_.set_tooltip(
50- /** TRANSLATORS: Editor set resources access key. **/
51- (boost::format(_("Use %s to set to this value"))
52- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
53- % _("Ctrl + Click")).str());
54+ /** TRANSLATORS: Editor set rseources access key. **/
55+ _("Ctrl + Click on the map to set the amount of the selected resource"));
56
57 change_by_.changed.connect
58 (boost::bind
59
60=== modified file 'src/editor/ui_menus/tool_menu.cc'
61--- src/editor/ui_menus/tool_menu.cc 2016-04-06 09:23:04 +0000
62+++ src/editor/ui_menus/tool_menu.cc 2016-04-08 07:36:16 +0000
63@@ -64,7 +64,7 @@
64 pos.x += width + spacing; \
65
66 ADD_BUTTON("change_height", _("Change height"));
67- ADD_BUTTON("noise_height", _("Noise height"));
68+ ADD_BUTTON("noise_height", _("Random height"));
69 ADD_BUTTON("set_terrain", _("Terrain"));
70 ADD_BUTTON("place_immovable", _("Immovables"));
71 ADD_BUTTON("place_bob", _("Animals"));
72
73=== modified file 'src/editor/ui_menus/tool_noise_height_options_menu.cc'
74--- src/editor/ui_menus/tool_noise_height_options_menu.cc 2016-04-07 06:06:42 +0000
75+++ src/editor/ui_menus/tool_noise_height_options_menu.cc 2016-04-08 07:36:16 +0000
76@@ -21,8 +21,6 @@
77
78 #include <cstdio>
79
80-#include <boost/format.hpp>
81-
82 #include "base/i18n.h"
83 #include "editor/editorinteractive.h"
84 #include "editor/tools/decrease_height_tool.h"
85@@ -40,7 +38,7 @@
86 EditorNoiseHeightTool & noise_tool,
87 UI::UniqueWindow::Registry & registry)
88 :
89- EditorToolOptionsMenu(parent, registry, 300, 120, _("Noise Height Options")),
90+ EditorToolOptionsMenu(parent, registry, 300, 120, _("Random Height Options")),
91 noise_tool_(noise_tool),
92 box_(this, hmargin(), vmargin(), UI::Box::Vertical, 0, 0, vspacing()),
93 lower_(&box_, 0, 0, get_inner_w() - 2 * hmargin(), 80,
94@@ -60,16 +58,14 @@
95 UI::SpinBox::Type::kSmall)
96 {
97 lower_.set_tooltip(
98- /** TRANSLATORS: Editor noise height access keys. **/
99- _("Click to set the height to a random value within the specified range"));
100+ /** TRANSLATORS: Editor random height access key. **/
101+ _("Click on the map to set terrain height to a random value within the specified range"));
102 upper_.set_tooltip(
103- /** TRANSLATORS: Editor noise height access keys. **/
104- _("Click to set the height to a random value within the specified range"));
105+ /** TRANSLATORS: Editor random height access key. **/
106+ _("Click on the map to set terrain height to a random value within the specified range"));
107 set_to_.set_tooltip(
108- /** TRANSLATORS: Editor set hoise height access keys. **/
109- (boost::format(_("Use %s to set to this value"))
110- /** TRANSLATORS: This is an access key combination. Localize, but do not change the key. **/
111- % _("Ctrl + Click")).str());
112+ /** TRANSLATORS: Editor set height access key. **/
113+ _("Ctrl + Click on the map to set terrain height"));
114
115 lower_.changed.connect
116 (boost::bind

Subscribers

People subscribed via source and target branches

to status/vote changes: