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

New feature: ignore new touches at top edge. #115

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mcourteaux
Copy link

@mcourteaux mcourteaux commented Jul 26, 2016

I'm using this driver on my MacBook Pro. Very often I accidentally clicked using the side of my thumbs when I'm typing on the keyboard. I only touched the top 10% or something of the touchpad. So I added a new feature to be able to ignore new touches on the top of the touchpad.

Two new variables: TopEdge and IgnoreTop. I know this could have been done in one setting, however, it seems to be more generic towards to future to split the size and the behavior of it.

Edit: After using this for some time. I decided to add support for specifically ignoring the top corners, instead of the top edge. This also introduces two new variables: TopCorners and IgnoreTopCorners. This also showed there was a bug in the coordinate system. You forgot to subtract the minimum value of the coordinates (obtained through the capabilities). So right now, I'm subtracting the minimum before storing the coordinates in the struct Touch its x and y variables. I'm not sure if this breaks anything else (since coordinates are now always non-negative), but so far I'm not experiencing any problems. If this does break something, please let me know!

Note: my vim stripped the trailing spaces in the README.md file. I can't really undo that (because I hate nano).

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.

1 participant