Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #1475 #1483

Merged
Merged

Conversation

Daniel-Esteban
Copy link
Contributor

@Daniel-Esteban Daniel-Esteban commented Aug 15, 2022

Before, CTRL+SHIFT keystrokes registered as like so:

CTRL+SHIFT+p = <C-P> instead of <C-S-p> or <C-S-P>
CTRL+SHIFT+x = <C-X> instead of <C-S-x> or <C-S-X>
...

So different keymaps couldn't be assigned to CTRL+x and CTRL+SHIFT+x
because nvim assumes it is the same mapping.

Now, CTRL+SHIFT+x is interpreted as <C-S-X>, so different mappings can
be assigned to CTRL+x and CTRL+SHIFT+x.

What kind of change does this PR introduce?

  • Fix

Did this PR introduce a breaking change?

A breaking change includes anything that breaks backwards compatibility either at compile or run time.

  • Maybe, I don't know

Before, CTRL+SHIFT keystrokes registered as like so:

CTRL+SHIFT+p = <C-P> instead of <C-S-p> or <C-S-P>
CTRL+SHIFT+x = <C-X> instead of <C-S-x> or <C-S-X>
...

So different keymaps couldn't be assigned to CTRL+x and CTRL+SHIFT+x
because nvim assumes it is the same mapping.

Now, CTRL+SHIFT+x is interpreted as <C-S-X>, so different mappings can
be assigned to CTRL+x and CTRL+SHIFT+x.
@MultisampledNight MultisampledNight linked an issue Aug 15, 2022 that may be closed by this pull request
@MultisampledNight
Copy link
Contributor

Thank you!

Even though this is technically a breaking change, since someone could've wanted to map Ctrl + Shift + ... and mapped it to <C-...> instead, but technically it just opens up a new usecase. So I wouldn't consider this a breaking change but a change worth documenting anyways.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ctrl+Shift keybinds not working properly <C-S-x>
2 participants