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 custom event type replacing crossterm's Event #3169

Merged
merged 1 commit into from Aug 9, 2022

Conversation

sudormrfbin
Copy link
Member

Add custom event type replacing crossterm's Event

Ported over from 61365df in the gui branch. This will allow adding our own events, most notably an idle timer event (useful for adding debounced input in dynamic pickers used by interactive global search and workspace symbols).

@groves
Copy link
Contributor

groves commented Jul 29, 2022

This could be nice for #3233. I'm doing some gyrations in crossterm-rs/crossterm#693 to keep Copy on its event type. That change could be simpler if Copy could be dropped in crossterm, and having a custom event type in Helix would make that easier.

@archseer
Copy link
Member

archseer commented Aug 5, 2022

Can you resolve the merge conflict here?

Ported over from 61365df in the `gui` branch. This will allow
adding our own events, most notably an idle timer event (useful
for adding debounced input in [dynamic pickers][1] used by interactive
global search and workspace symbols).

[1]: helix-editor#3110
@archseer archseer merged commit 634b6d4 into helix-editor:master Aug 9, 2022
@sudormrfbin sudormrfbin deleted the native-event-type branch August 9, 2022 03:45
thomasskk pushed a commit to thomasskk/helix that referenced this pull request Sep 9, 2022
Ported over from 61365df in the `gui` branch. This will allow
adding our own events, most notably an idle timer event (useful
for adding debounced input in [dynamic pickers][1] used by interactive
global search and workspace symbols).

[1]: helix-editor#3110

Co-authored-by: Blaž Hrastnik <blaz@mxxn.io>
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

3 participants