widelands-dev team mailing list archive

[Angelo Locritani <angelo.locritani@xxxxxxxxx>]

Angelo Locritani has proposed merging lp:~alocritani/widelands/switch_buttons_on_economy_options_window into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~alocritani/widelands/switch_buttons_on_economy_options_window/+merge/84900

Since we changed the position of +/- buttons in options windows (bug896879), I've switched the buttons in economy options windows also.
-- 
https://code.launchpad.net/~alocritani/widelands/switch_buttons_on_economy_options_window/+merge/84900
Your team Widelands Developers is requested to review the proposed merge of lp:~alocritani/widelands/switch_buttons_on_economy_options_window into lp:widelands.

=== modified file 'src/wui/transport_ui.cc'
--- src/wui/transport_ui.cc	2011-11-30 21:38:37 +0000
+++ src/wui/transport_ui.cc	2011-12-08 08:06:45 +0000
@@ -143,9 +143,9 @@
 		  text, tooltip, m_can_act);                                \
 	b->sigclicked.connect(boost::bind(&Economy_Options_Ware_Panel::callback, this)); \
 	buttons->add(b, UI::Box::AlignCenter);
-			ADD_WARE_BUTTON(increase_target, "+", _("Increase target"))
+			ADD_WARE_BUTTON(decrease_target, "-", _("Decrease target"))			
 			b->set_repeating(true);
-			ADD_WARE_BUTTON(decrease_target, "-", _("Decrease target"))
+			ADD_WARE_BUTTON(increase_target, "+", _("Increase target"))			
 			b->set_repeating(true);
 			ADD_WARE_BUTTON(reset_target, "R", _("Reset to default"))
 		}
@@ -238,10 +238,10 @@
 	b->sigclicked.connect(boost::bind(&Economy_Options_Worker_Panel::callback, this)); \
 	buttons->add(b, UI::Box::AlignCenter);
 
+			ADD_WORKER_BUTTON(decrease_target, "-", _("Decrease target"))
+			b->set_repeating(true);
 			ADD_WORKER_BUTTON(increase_target, "+", _("Increase target"))
 			b->set_repeating(true);
-			ADD_WORKER_BUTTON(decrease_target, "-", _("Decrease target"))
-			b->set_repeating(true);
 			ADD_WORKER_BUTTON(reset_target, "R", _("Reset to default"))
 		}