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

Improve keycode merging #221

Merged

Conversation

rene-descartes2021
Copy link
Collaborator

@rene-descartes2021 rene-descartes2021 commented Aug 17, 2022

Rather than list specific cases <Tab> <Space>, I think it better to generalize.

This PR proposes a way to generalize the mapping of keycodes to internal data-structures (description dictionary, native cache).

Also a couple other improvements:

  1. Improved regex escaping in s:has_children()
  2. Improved loop in s:wait_with_timeout(), there was a 20ms window before the timeout where if the user pressed a key in that timespan it would be considered not pressed.

@rene-descartes2021 rene-descartes2021 marked this pull request as ready for review August 17, 2022 17:24
@liuchengxu liuchengxu merged commit 9875bae into liuchengxu:master Oct 25, 2022
@liuchengxu
Copy link
Owner

Sorry for the late merge, great work! 🍻

@rene-descartes2021 rene-descartes2021 deleted the keycodes-improvement branch October 26, 2022 21:09
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.

2 participants