Normalize Keymap Capitalization

[barrett-ruth]

After debugg my issue here I found out that mapping keymaps with lowercase codes (i.e. <c-space>, not <C-space>) was broken.

The current keymapping logic simply overrides the default preset by overwriting exact matches - in this case, ignoring all my keybindings

(Neo)Vim supports arbitrary capitalization for keymaps (<Up>, <up>), etc. so I figured blink should too.

[Saghen]

Thanks!

[barrett-ruth]

ty for the correction :)

[milanglacier]

Hi,

I think we should not capitalize all the keys arbitrarily.

For example <A-a> and <A-A> are two different keys. <A-A> is actually alt+shift+a.

Besides, if the user binds the key to a two keys sequence, like <c-k>Y, this will be a breaking change.

In fact <C-space> is not a regular terminal supported key binding (but <A-space> is a regular supported keybinding). Some terminals like kitty support it. But we should not assume the users are using a terminal that would support this key. So I am not surprised if we see any unexpected behavior with this key.

[Saghen]

@milanglacier Can't believe I didn't think about that, sorry! Fixed on main by comparing the escaped keycodes

[milanglacier]

@milanglacier Can't believe I didn't think about that, sorry! Fixed on main by comparing the escaped keycodes

No problem. We human make mistakes. Thanks for your great effort working on this project!

[barrett-ruth]

SMH. Should've considered all possible keycodes. Thanks so much for the correction. In me finding quirky keymappings... I forgot about the even quirkier ones. Thanks @milanglacier and @Saghen !!!!