Alt+Enter newline shortcut

[goolord]

in most messaging apps, shift+enter enters a newline character. this pr adds that behavior.
currently i'm unable to test if this behaves as intended because running gurk panics for me

[boxdot]

Thank you! I had the same implementation but it does not work: crossterm-rs/crossterm#400.

off-topic: If you have the line number where it panics, I might be able to fix it.

[goolord]

I'll create a seperate issue about the panic.
What about Alt+Enter? That's how my terminal allows entering newlines

edit: and the sequence is supported

edit 2: ah, the panic mentioned earlier is solved at #7

[boxdot]

What about Alt+Enter? That's how my terminal allows entering newlines

Sounds good!

[goolord]

Ok, I write very little rust but hopefully nothing looks off. The solution is a little janky, crossterm reads Alt+Enter as '\r', so all of the logic is actually in draw_chat. Even though this eliminated the need for the key modifiers, I left the put_char function factored out because I think it's nice.

[boxdot]

Ok, I write very little rust but hopefully nothing looks off.

No worries. :) I appreciate the contribution a lot.

The solution is a little janky, crossterm reads Alt+Enter as '\r', so all of the logic is actually in draw_chat. Even though this eliminated the need for the key modifiers, I left the put_char function factored out because I think it's nice.

I like the factored out function. I was thinking that now, in the current implementation, we send \r to the Signal server. I guess we could replace \r just by \n in the input field on input? Then there is no need for special handling in ui, right?

[goolord]

I'm not familiar enough with tui to know why this is, but the '\n' (and '\r') character doesn't actually visually create a newline, instead each index in lines: Vec<String> is rendered as a new line.. it ends up being fine I think because it helps in calculating the y position of the input cursor. Also, I just pushed a commit which fixes a few bugs that were introduced by rendering '\n' characters

[boxdot]

Private life got me, new job, etc... ☔ I am really sorry for being not responsive.

[boxdot]

@goolord I hope you are fine with me changing this behavior. With #131, Alt+Enter will be remapped to switch between the multi-line and single-line modes.