widelands-dev team mailing list archive

[Angelo Locritani <angelo.locritani@xxxxxxxxx>]

Angelo Locritani has proposed merging lp:~alocritani/widelands/even_more_on_+_- into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~alocritani/widelands/even_more_on_+_-/+merge/85362

Fixed some CodeCheck warning and added a space between +/- and Reset button in economy options window
-- 
https://code.launchpad.net/~alocritani/widelands/even_more_on_+_-/+merge/85362
Your team Widelands Developers is requested to review the proposed merge of lp:~alocritani/widelands/even_more_on_+_- into lp:widelands.

=== modified file 'src/wui/transport_ui.cc'
--- src/wui/transport_ui.cc	2011-12-06 20:54:23 +0000
+++ src/wui/transport_ui.cc	2011-12-12 16:56:31 +0000
@@ -143,10 +143,11 @@
 		  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(decrease_target, "-", _("Decrease target"))			
-			b->set_repeating(true);
-			ADD_WARE_BUTTON(increase_target, "+", _("Increase target"))			
-			b->set_repeating(true);
+			ADD_WARE_BUTTON(decrease_target, "-", _("Decrease target"))
+			b->set_repeating(true);
+			ADD_WARE_BUTTON(increase_target, "+", _("Increase target"))
+			b->set_repeating(true);
+			buttons->add_space(8);
 			ADD_WARE_BUTTON(reset_target, "R", _("Reset to default"))
 		}
 
@@ -242,6 +243,7 @@
 			b->set_repeating(true);
 			ADD_WORKER_BUTTON(increase_target, "+", _("Increase target"))
 			b->set_repeating(true);
+			buttons->add_space(8);
 			ADD_WORKER_BUTTON(reset_target, "R", _("Reset to default"))
 		}