-
Notifications
You must be signed in to change notification settings - Fork 7
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
Switch from pynput to libvinput #134
Conversation
CI needs to be re-run because when I made my latest commit, 1.4 was not uploaded to PyPi, since I was doing development locally to be able to debug the library |
f242250
to
6639305
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think @Raymo111 should take care of the installer, I am not familiar with how PyInstaller works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested both the dist
and installed versions on Windows and all seem to be functional.
SO nice to not have to wait for the start logging so long with the old freezing, lol.
macOS broke again im gonna cry |
171fad2
to
fa33959
Compare
Both Linux (X11) and macOS work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I re-tested to make sure that the mouse stuff was fixed, it looks good to me.
eb4d2aa
to
24ccae6
Compare
Signed-off-by: Slendi <[email protected]>
This is an easier approach IMHO. Signed-off-by: Slendi <[email protected]>
Signed-off-by: Slendi <[email protected]>
Signed-off-by: Slendi <[email protected]>
Signed-off-by: Slendi <[email protected]>
Signed-off-by: Slendi <[email protected]>
Signed-off-by: Slendi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 more to go
Signed-off-by: Slendi <[email protected]>
Signed-off-by: Raymond Li <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved provided testing works
Edge case where a non-allowed char key (e.g. number) typed fast enough to be considered a chord is added as the first character of a word, and timing is not initialized. When a non-chord key (e.g. modifier key) is then pressed (to end the word), the word is logged but there is no timing associated with it, which would cause a crash.
I need to update libvinput first to 1.4 so more types are available outside of the CFFI.