GNOME Bugzilla – Bug 757026
Add scroll source / axis stop information to scroll events
Last modified: 2016-01-14 18:16:39 UTC
On the evdev backend we can fetch already this information from libinput, on x11 we could infer some of this info, although I couldn't really tell for how long... This may come useful in order to implement the (quite similar to libinput's) support for wayland: https://patchwork.freedesktop.org/patch/61985/ In the patches I'm attaching, I've added the fields to ClutterScrollEvent itself, according to pahole those still fit, although the struct is already on the limit, could it be preferred to put these into private data?
Created attachment 313946 [details] [review] clutter-event: Add scroll source enum and axis scroll flags Those can be used to implement different scrolling behaviors. The fields have been added to ClutterScrollEvent itself. According to pahole, this makes the struct as big as ClutterButtonEvent and ClutterTouchEvent, so already at the limit of the ClutterEvent union.
Created attachment 313947 [details] [review] evdev: Add scroll source/finish flags info to scroll events. Libinput tells us this much, so make use of it.
Review of attachment 313946 [details] [review]: Okay.
Review of attachment 313947 [details] [review]: Okay.
Attachment 313946 [details] pushed as 89c1c1e - clutter-event: Add scroll source enum and axis scroll flags Attachment 313947 [details] pushed as 307cb58 - evdev: Add scroll source/finish flags info to scroll events.