Merge lp:~widelands-dev/widelands/bug-1699778-multilineeditbox-handle-key-enter into lp:widelands

[Jukka Pakarinen]

Fix enter key handling in multilineeditbox. The cursor would remain in the old line after enter was pressed, now it's moving to the next line as expected.

The change in the branch fixes the bug 1699778. update() function sets the ww_valid flag off which means that multilineeditbox must be refreshed. Without the function call refresh will happen during a next call of some other key which sets the flag off.

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 2928. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/311413193.
Appveyor build 2737. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1699778_multilineeditbox_handle_key_enter-2737.

[GunChleoc (gunchleoc)]

Great fix, thanks!

@bunnybot merge

review: Approve

=== modified file 'src/ui_basic/multilineeditbox.cc'
--- src/ui_basic/multilineeditbox.cc	2017-09-20 21:27:25 +0000
+++ src/ui_basic/multilineeditbox.cc	2017-12-04 16:56:11 +0000
@@ -375,6 +375,7 @@
 		case SDLK_KP_ENTER:
 		case SDLK_RETURN:
 			d_->insert(d_->cursor_pos, "\n");
+			d_->update();
 			changed();
 			break;