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

Add shift-backspace keybind alias for backspace #4937

Merged
merged 1 commit into from
Nov 29, 2022
Merged

Add shift-backspace keybind alias for backspace #4937

merged 1 commit into from
Nov 29, 2022

Conversation

lesleyrs
Copy link
Contributor

Not sure if this PR works but would be nice to have, no real reason not to I guess?

@the-mikedavis
Copy link
Member

By default the terminal can't tell the difference between backspace and S-backspace: all of these commands already work with S-backspace. Terminals that implement the progressive keyboard enhancement protocol can tell the difference but we need to detect and enable that from within Helix. See the enable-keycode-disambiguation branch. We could merge this change into that branch.

@lesleyrs lesleyrs changed the base branch from master to enable-keycode-disambiguation November 29, 2022 18:12
@lesleyrs
Copy link
Contributor Author

Ok sounds good, I'm not sure how to have 2 separate helix installs work so I can't test if I did it correctly. And I don't wanna mess with my current install.

@gabydd
Copy link
Member

gabydd commented Nov 29, 2022

Just FYI enable key code disambiguation was just merged also you can do cargo build or cargo run to test it out

@the-mikedavis the-mikedavis force-pushed the enable-keycode-disambiguation branch from b31b320 to a57e5ed Compare November 29, 2022 19:07
@the-mikedavis
Copy link
Member

#4887 handles the new keys but doesn't enable the disambiguation flags. See #4887 (comment)

@gabydd
Copy link
Member

gabydd commented Nov 29, 2022

Oh right sorry about that.

@kirawi kirawi closed this Nov 29, 2022
@kirawi kirawi reopened this Nov 29, 2022
@kirawi kirawi added A-keymap Area: Keymap and keybindings S-waiting-on-review Status: Awaiting review from a maintainer. labels Nov 29, 2022
@kirawi
Copy link
Member

kirawi commented Nov 29, 2022

Misclick.

@lesleyrs
Copy link
Contributor Author

Ok I just did a cargo run and it seems to work as intended in both editor and prompt 👍. Thx for the help guys!

@the-mikedavis the-mikedavis changed the title Add shift-backspace keybind in addition to just backspace Add shift-backspace keybind alias for backspace Nov 29, 2022
@the-mikedavis the-mikedavis merged commit 4a4b1de into helix-editor:enable-keycode-disambiguation Nov 29, 2022
@lesleyrs lesleyrs deleted the shift-backspace branch November 30, 2022 09:42
the-mikedavis pushed a commit that referenced this pull request Feb 11, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
the-mikedavis pushed a commit that referenced this pull request Feb 26, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
the-mikedavis pushed a commit that referenced this pull request Feb 26, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
archseer pushed a commit that referenced this pull request Feb 28, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
estin pushed a commit to estin/helix that referenced this pull request Mar 4, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
wes-adams pushed a commit to wes-adams/helix that referenced this pull request Jul 4, 2023
When the Kitty Keyboard Protocol is enabled, S-backspace is
distinguished from backspace with no modifiers. This is awkward when
typing because it's very easy to accidentally hold shift and press
backspace temporarily when typing capital letters.

Kakoune (which is also a Kitty Keyboard Protocol application) treats
S-backspace as backspace too:
https://github.com/mawww/kakoune/blob/3150e9b3cd8e61d9bc68245d67822614d4376cf4/src/input_handler.cc#L1275
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-keymap Area: Keymap and keybindings S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants