Allow Ctrl + arrow keys sentence navigation in commit message

[mjarkk]

Describe the bug
Can't change the cursor position in a commit message using Ctrl + arrow keys (← →)
Currently when doing this the popup changes to:

┌─Clear file panel────────────────────────────────────┐
│Are you sure you want `reset --hard HEAD`? You may lo│
│se changes                                           │
└─────────────────────────────────────────────────────┘

To Reproduce
Steps to reproduce the behavior:

1. Open lazygit
2. Under commits Press C
3. Do: Ctrl + ← OR Ctrl + →
4. See the popup change

Expected behavior
When the cursor is here in a sentence: some words with| a cursor (| = the cursor)
And pressed Ctrl + ← the cursor needs to be here: some words |with a cursor

Desktop:

• OS: Linux (fedora 4.17.12-200.fc28.x86_64)
• Shell: Bash
• Lazygit v0.1.59

Additional context
I'm not sure if Ctrl + ← is possible in Go.
If not it might be a good idea to block the Clear files panel popup

[jesseduffield]

looks like it's not possible: nsf/termbox-go#60
It will always be interpreted as the beginning of an escape sequence hence the weird behaviour.

[jesseduffield]

closing cos it seems like this isn't possible :P

[asfaltboy]

@jesseduffield I know it's been just shy of 4 years, but I wonder is it still impossible, given we've switched to tcell (albeit a fork)? Would we be able to add a custom binding to implement this behavior manually if not?

For me, at least, jumping between words to fix typos is a productivity super-power 🙏

[stefanhaller]

Jumping by words was implemented in #2204, I use it all the time and could hardly live without it. But it uses alt (option on Mac) instead of ctrl.

[asfaltboy]

Thanks for the quick reply @stefanhaller, this was probably a user error!

I'm using kitty terminal emulator on macOS (and also in Arch) but pressing alt+left/right in lazygit only added the b or f character to the output.

But now I remember that I added a few mappings to allow alt+left/right word jumping in edgedb-cli.

Commenting these out solved it for me:

 # jump word in apps like edgedb
-map alt+left send_text all \x1b\x62
-map alt+right send_text all \x1b\x66
+# map alt+left send_text all \x1b\x62
+# map alt+right send_text all \x1b\x66

This means I'll have to figure out another way to send the right sequence to edgedb-cli (uses a rust readline lib, see source)... 😢