widelands-dev team mailing list archive

[GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>]

GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1418050 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1418050 in widelands: "Secondary and Third Alternative Tool no longer reset to the first tool if you let go of the button"
  https://bugs.launchpad.net/widelands/+bug/1418050

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1418050/+merge/258913

Added handling of SDL_KEYUP event. This fixes that the editor did not return to the first tool when SHIFT or ALT etc. was released.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1418050 into lp:widelands.

=== modified file 'src/wlapplication.cc'
--- src/wlapplication.cc	2015-03-01 09:21:20 +0000
+++ src/wlapplication.cc	2015-05-12 17:37:44 +0000
@@ -550,6 +550,7 @@
 	SDL_Event ev;
 	while (poll_event(ev)) {
 		switch (ev.type) {
+		case SDL_KEYUP:
 		case SDL_KEYDOWN: {
 			bool handled = false;
 			if (cb && cb->key) {