teamikl/wahjam — comment on commit 07888e647ee68edc33e53fa21841f6da47f2406c
File: ninjam/qtclient/MainWindow.cpp line 585
By teamikl:

self-review

1) I wanted to send MSG to server, but I did not want to write or copy the same code from ChatInputReturnPressed.
TODO: separate common code to send message public method that common use.

2) insertText on other methods can break text struct.

append will add text on the End, it is ok.
but insertText looks assume the cursor position was the End.
every time before insertText should ensure the cursor position.

3) maybe no needed "emit", just call the slot directly for this case.