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

[x11] Add support for tablet device detection #105

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

exLeute
Copy link

@exLeute exLeute commented Sep 5, 2024

Fix tablet detection system by checking for device axes instead of string comparison.

@exLeute
Copy link
Author

exLeute commented Sep 5, 2024

@dacap sorry for late pr, don't really have time, but main logic should be ready, also i think i have not added general settings to manually put tablet, but it still should work better than detecting tablet by string, because i could not debug this functionality. But main detection thing works on both of my x11 systems as intended.

@exLeute exLeute changed the title [x11] Add support for user-defined tablet device pattern detection [x11] Add support for tablet device detection Sep 5, 2024
@dacap dacap self-assigned this Sep 11, 2024
@dacap
Copy link
Member

dacap commented Sep 13, 2024

Hi @exLeute, not sure if there are other devices with more than 3 axis that are not a stylus (e.g. a mouse with horizontal scroll). So not sure if this would work for all cases.

@dacap dacap removed their assignment Sep 13, 2024
@exLeute
Copy link
Author

exLeute commented Sep 13, 2024

True @dacap, but it can be polished and added as an optional detection method, or something like drop down list of all "xinput" devices

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants