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

Ignore key-release keyboard events #6139

Merged
merged 1 commit into from
Mar 1, 2023

Conversation

the-mikedavis
Copy link
Member

Since crossterm 0.26.x, we receive press/release keyboard events on Windows always. We can ignore the release events though to emulate the behavior of keyboard input on Windows on crossterm 0.25.x.

Fixes #6133

Since crossterm 0.26.x, we receive press/release keyboard events on
Windows always. We can ignore the release events though to emulate
the behavior of keyboard input on Windows on crossterm 0.25.x.
@the-mikedavis the-mikedavis added C-bug Category: This is a bug E-testing-wanted Call for participation: Experimental features suitable for testing labels Feb 28, 2023
@the-mikedavis the-mikedavis mentioned this pull request Feb 28, 2023
@the-mikedavis the-mikedavis added S-waiting-on-review Status: Awaiting review from a maintainer. E-testing-wanted Call for participation: Experimental features suitable for testing and removed E-testing-wanted Call for participation: Experimental features suitable for testing labels Feb 28, 2023
@the-mikedavis the-mikedavis marked this pull request as ready for review February 28, 2023 18:52
@archseer archseer merged commit 27211ab into master Mar 1, 2023
@archseer archseer deleted the fix/windows/duplicate-key-events branch March 1, 2023 01:26
estin pushed a commit to estin/helix that referenced this pull request Mar 4, 2023
Since crossterm 0.26.x, we receive press/release keyboard events on
Windows always. We can ignore the release events though to emulate
the behavior of keyboard input on Windows on crossterm 0.25.x.
wes-adams pushed a commit to wes-adams/helix that referenced this pull request Jul 4, 2023
Since crossterm 0.26.x, we receive press/release keyboard events on
Windows always. We can ignore the release events though to emulate
the behavior of keyboard input on Windows on crossterm 0.25.x.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug E-testing-wanted Call for participation: Experimental features suitable for testing S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

typing twice.
2 participants