Skip to content

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