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: check key event kind for the simple template #55

Merged
merged 3 commits into from
Apr 29, 2024

Conversation

vm910
Copy link
Contributor

@vm910 vm910 commented Apr 29, 2024

Hello!

I stumbled onto this crate by accident and wanted to make a toy in it. After making it, I tested it on Linux and Windows and noticed that it produces double inputs on Windows after every key stroke. I saw in the FAQ what the problem was and thought it'd be best to patch this here. The simple-async template seems to already have this check there, so there's no need to touch that I think.

This is my first ever PR so I apologize in advance if I made any mistakes.

Copy link
Sponsor Member

@orhun orhun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

simple/Cargo.toml Outdated Show resolved Hide resolved
simple/src/main.rs Outdated Show resolved Hide resolved
Copy link
Sponsor Member

@orhun orhun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@orhun orhun changed the title key event kind check on the simple template fix: check key event kind for the simple template Apr 29, 2024
@orhun orhun merged commit 538de88 into ratatui-org:main Apr 29, 2024
3 checks passed
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.

None yet

2 participants