LGTM. 2 issues though: 1. Don't forget to remove the text conflict in src/ui_basic/panel.h
2. Alt switches the tool, but clicking doesn't do anything. I assume it's the Ubuntu window manager getting in the way. -- https://code.launchpad.net/~widelands-dev/widelands/ui_fixes/+merge/242561 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ui_fixes into lp:widelands. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
