I wonder if that is related to the upgrade of tcell. Depending on your terminal, ^h may send the backspace code.

I have the same issue. My terminal is alacritty, fish_key_reader reports the following:

$ fish_key_reader
Press a key:
              hex:    8  char: \cH  (or \b)
bind -k backspace 'do something'
bind \b 'do something'

Downgrading tcell from v2.4.0 to v2.0.0 does not seem to fix the issue. Any ideas what else could cause this regression?

Do you reproduce the issue with another terminal emulator? If not, maybe this is related to alacritty terminfo?

I can reproduce the error in Konsole and kitty as well.

I added this line just above line 243 in widgets/aerc.go

aerc.logger.Printf("bindings.GetBinding(%v)", aerc.pendingKeys)

The codes I got for ctrl+h were {0 8 8} (no modifier, key=8, rune=8) which does indeed look like a backspace.

I noticed config/bindings.go had a list of keys and then tried binding <bs> - which seemed to work for me. If that is the same for the other commenters then we have a simple work-around at least.

Could the behaviour have changed back when the modifier was added to the KeyStroke struct? The tcell change just being a coincidence?

Yes, indeed binding <bs> = :prev-tab<Enter> allows moving back to a prev tab with <C-h>

Same here

Robin Jarry referenced this ticket in commit b963265.

Robin Jarry referenced this ticket in commit b963265.

Robin Jarry referenced this ticket in commit b963265.

Robin Jarry referenced this ticket in commit b963265.