widelands-dev team mailing list archive

[GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>]

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

Commit message:
Fixed update bug in Multiline Editbox.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1540782 in widelands: "Multiline edit box shows new text only when hitting backspace"
  https://bugs.launchpad.net/widelands/+bug/1540782

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1540782-multilineeditbox/+merge/287590

Found the correct update function to call, text input in Map Options should be fine now :)
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1540782-multilineeditbox into lp:widelands.

=== modified file 'src/ui_basic/multilineeditbox.cc'
--- src/ui_basic/multilineeditbox.cc	2016-02-08 00:18:02 +0000
+++ src/ui_basic/multilineeditbox.cc	2016-03-01 08:34:46 +0000
@@ -419,6 +419,7 @@
 	if (d_->text.size() + input_text.size() <= d_->maxbytes) {
 		d_->insert(d_->cursor_pos, input_text);
 		changed();
+		d_->update();
 	}
 	return true;
 }