On Fri, Jul 31, 2015 at 1:43 AM, Hans de Goede <[email protected]> wrote:
> > I thought about that when doing the patch but decided for this approach on >> purpose. currently, a mis-detection of a pinch gesture with gestures >> enabled means the gesture remains until the fingers are lifted. with this >> patch otoh it just doesn't do anything until we detect a scroll gesture, >> at >> which point we switch to scrolling. >> > > I see that is a valid point. > > >> the point here is that the scroll detection code is still at work, so >> hopefully that "trains" users to perform the right scroll action. we get >> consistent behaviour across touchpads too this way, and we avoid the >> situations down the track of "it used to scroll this way but now it >> doesn't >> anymore" when we add other features, or when we manage do to gestures >> reliably on those touchpads after all. >> >> I might be overthinking this though :) >> > > You are :) > > But I can see the logic in not training users to use the "wrong" kind of > 2fg > gestures for scrolling, so lets go with your approach: > Could the scroll gesture "cancel" the pinch gesture when it is detected? Or is cancelling only done by the compositor and not by libinput.
_______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
