Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/soldierselect_radiobutton into lp:widelands

2013-07-27 Thread Teppo Mäenpää
typo: Without this check - Without this patch -- https://code.launchpad.net/~widelands-dev/widelands/soldierselect_radiobutton/+merge/176784 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/soldierselect_radiobutton.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/soldierselect_radiobutton into lp:widelands

2013-07-27 Thread SirVer
I agree that the new buttons look really good, and that their placement is good as well. Thanks! Glad that you like it that way too. However, start and stop buttons have now appeared to the lower left corner. Good catch - I messed that up. I added something along the lines of your patch and

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/soldierselect_radiobutton into lp:widelands

2013-07-25 Thread cghislai
Review: Approve Looks good and works great. I also like those new icons. Should there be such option for training sites as well, so we can choose to train rookies or heroes ? I have no idea if the underlying logic could allow this --

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/soldierselect_radiobutton into lp:widelands

2013-07-25 Thread Teppo Mäenpää
About trainingsites: How should these options differ? Currently, there is an option regarding whether the trainingsite attempts to train least-trained guy or most-trained guy, if there are more than one tha could be trained in a step. There is no UI for that. Also, it would be relatively easy