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

Added pinch gestures #29

Closed
wants to merge 3 commits into from
Closed

Added pinch gestures #29

wants to merge 3 commits into from

Conversation

osleg
Copy link
Collaborator

@osleg osleg commented Oct 27, 2019

Basic implementation enables one shot pinch gesture.
That is the command from config file will run only once as soon as
fingers move 50% the distance between the fingers.

osleg and others added 3 commits October 27, 2019 19:30
Basic implementation enables one shot pinch gesture.
That is the command from config file will run only once as soon as
fingers move 50% the distance between the fingers.
I felt that pinch gesture is kinda tricky to execute on touchpad,
so I changed a way of calculation when to trigger the command and
added the distance for fingers travel to configuration.
Basically a `0.5` distance feels quite ok to me, but it becomes really
snappy at `0.1`.
Added a configurable distance variable to pinch gesture.
@yigitsever
Copy link

I've been anticipating the pinch gestures since #10 was submitted. Does this answer the concerns given there?

@osleg
Copy link
Collaborator Author

osleg commented Oct 30, 2019

@yigitsever in my current implementation I ignore fingers count as this was mainly implemented just for myself and all I need is punch in and out.

Anyway I'll try to look into that, no promises tho :-)

@osleg
Copy link
Collaborator Author

osleg commented Oct 30, 2019

@yigitsever per what I see from my test fingers detection is quite reliable and mistakes are rare.
Could be that different hardware would act differently. I will make new PRs with everything organized.

Fingers count: 2
Fingers count: 2
Fingers count: 2
Fingers count: 2
Fingers count: 3
Fingers count: 3
Fingers count: 4
Fingers count: 3
Fingers count: 3
Fingers count: 4
Fingers count: 4
Fingers count: 4
Fingers count: 4

@osleg
Copy link
Collaborator Author

osleg commented Oct 30, 2019

angle delta also works but it is a bit tricky, i'll ignore it for now

@osleg
Copy link
Collaborator Author

osleg commented Oct 30, 2019

Closing in favor of #30

@osleg osleg closed this Oct 30, 2019
@osleg osleg mentioned this pull request Oct 31, 2019
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