diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2017-05-09 15:07:20 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2017-05-11 10:30:04 +1000 |
commit | 211bed2c25e4282820e5155191d4dc09201f1210 (patch) | |
tree | 703698bd0c4102f3368026cd22837a8fa84b09c9 /test | |
parent | 368006ef40673125228a6c0f7b0528a4ecff0487 (diff) |
touchpad: pull the tap exclusion zone down to the full edge zone
This was originally left outside of the button areas in case users tap in
those zones, but we're getting false tap events in that zone.
On a 100mm touchpad, the edge zone is merely 5mm, it's acceptable to ignore
taps in that area even in the software button. We can revisit this if we see
tap detection failures in the future.
https://bugzilla.redhat.com/show_bug.cgi?id=1415796
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'test')
-rw-r--r-- | test/test-touchpad.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/test-touchpad.c b/test/test-touchpad.c index 2731500..d91c244 100644 --- a/test/test-touchpad.c +++ b/test/test-touchpad.c @@ -1176,15 +1176,15 @@ START_TEST(touchpad_palm_detect_tap_softbuttons) litest_drain_events(li); - litest_touch_down(dev, 0, 95, 5); + litest_touch_down(dev, 0, 99, 99); litest_touch_up(dev, 0); litest_assert_empty_queue(li); - litest_touch_down(dev, 0, 5, 5); + litest_touch_down(dev, 0, 1, 99); litest_touch_up(dev, 0); litest_assert_empty_queue(li); - litest_touch_down(dev, 0, 5, 99); + litest_touch_down(dev, 0, 10, 99); litest_touch_up(dev, 0); litest_assert_button_event(li, BTN_LEFT, @@ -1194,7 +1194,7 @@ START_TEST(touchpad_palm_detect_tap_softbuttons) LIBINPUT_BUTTON_STATE_RELEASED); litest_assert_empty_queue(li); - litest_touch_down(dev, 0, 95, 99); + litest_touch_down(dev, 0, 90, 99); litest_touch_up(dev, 0); litest_assert_button_event(li, BTN_LEFT, |