Commit 093a8789 authored by Peter Hutterer's avatar Peter Hutterer

touchpad: only identify for pinch in a distinct pinch position

Previously, any lower finger spaced more than the vertical threshold apart
would be labelled as thumb. This causes some taps to be detected as
single-taps, particularly where the user's hand is at an angle that causes the
touches to be effectively vertical.

Restructure that condition so that we only go for a thumb if we're
distinctively apart, and we only *not* go for thumb if we're distinctively
close together.

Fixes #359Signed-off-by: Peter Hutterer's avatarPeter Hutterer <>
(cherry picked from commit 0ce06d13)
parent ad7708f6
......@@ -335,15 +335,18 @@ tp_thumb_update_multifinger(struct tp_dispatch *tp)
/* Position-based thumb detection: When a new touch arrives, check the
* two lowest touches. If they qualify for 2-finger scrolling, clear
* thumb status. If not, mark the lower touch (based on pinch_eligible)
* as either PINCH or SUPPRESSED.
* thumb status.
* If they were in distinct diagonal position, then mark the lower
* touch (based on pinch_eligible) as either PINCH or SUPPRESSED. If
* we're too close together for a thumb, lift that.
if (mm.y > SCROLL_MM_Y) {
if (mm.y > SCROLL_MM_Y && mm.x > SCROLL_MM_X) {
if (tp->thumb.pinch_eligible)
tp_thumb_pinch(tp, first);
tp_thumb_suppress(tp, first);
} else {
} else if (mm.x < SCROLL_MM_X && mm.y < SCROLL_MM_Y) {
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment