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

2016-09-14 Thread Klaus Halfmann
Review: Approve compile test That fixed #1617576 for me, Launchpad does not show me the sources yet, so Ill try to check the code later. -- https://code.launchpad.net/~widelands-dev/widelands/local_functions/+merge/305415 Your team Widelands Developers is subscribed to branch

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/validate-translations into lp:widelands

2016-09-14 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/validate-translations into lp:widelands. Commit message: Added bash script for validating translations. Requested reviews: Widelands Developers (widelands-dev) For more details, see:

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

2016-09-14 Thread GunChleoc
I already did the code review, so this can go in :) @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/local_functions/+merge/305415 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/local_functions.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1616661_messagebox_hotkeys into lp:widelands

2016-09-14 Thread GunChleoc
Review: Resubmit Hmmm, people usually associate F1 with help, F5 with refresh etc... also, 0 has some nice semantics that we can't have with the function keys. I have found a workaround that will work for everything except toggling a button on and off with the same key while the game is

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/validate-translations into lp:widelands

2016-09-14 Thread bunnybot
Continuous integration builds have changed state: Travis build 1326. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/160009151. Appveyor build 1168. State: success. Details: