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

Changes to allow Windows backspace keybindings #422

Closed
wants to merge 12 commits into from
Closed

Changes to allow Windows backspace keybindings #422

wants to merge 12 commits into from

Conversation

fdncred
Copy link

@fdncred fdncred commented Jul 28, 2020

Added the ability to use Ctrl+Backspace, Meta+Backspace, and Shift+Backspace as keybindings on Windows. Thanks for your help @gwenn. I figure you'll have changes and not like some of the things I've done. :) I didn't run cargo fmt --all because I'm not sure if that's your practice.

examples/example.rs Outdated Show resolved Hide resolved
examples/example.rs Outdated Show resolved Hide resolved
src/tty/windows.rs Outdated Show resolved Hide resolved
src/tty/windows.rs Outdated Show resolved Hide resolved
src/keys.rs Outdated Show resolved Hide resolved
@fdncred
Copy link
Author

fdncred commented Jul 29, 2020

I believe I changed all your requested items. Please let me know if you have further feedback.

@fdncred fdncred requested a review from gwenn July 29, 2020 19:04
src/tty/windows.rs Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@fdncred fdncred requested a review from gwenn July 29, 2020 19:54
src/tty/windows.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@fdncred fdncred requested a review from gwenn August 10, 2020 18:04
@fdncred
Copy link
Author

fdncred commented Aug 26, 2020

@gwenn ping. are we ready to merge this?

@gwenn
Copy link
Collaborator

gwenn commented Aug 27, 2020

I would prefer to fix and merge #333 before.

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

Successfully merging this pull request may close these issues.

2 participants