After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 769554 - [Mir] Touchpad scrolling not as smooth as it should be
[Mir] Touchpad scrolling not as smooth as it should be
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Backend: Mir
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2016-08-05 12:41 UTC by Andreas Pokorny
Modified: 2016-09-28 15:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Do not filter out small touchpad scroll motions (1.64 KB, patch)
2016-08-05 12:41 UTC, Andreas Pokorny
none Details | Review
[PATCH] Do not filter out small scroll event fractions (1.55 KB, patch)
2016-08-13 23:32 UTC, Andreas Pokorny
committed Details | Review

Description Andreas Pokorny 2016-08-05 12:41:10 UTC
Created attachment 332799 [details] [review]
Do not filter out small touchpad scroll motions

When scrolling with a touchpad the left-right direction seems to be wrong, and it seems to be less smooth than with X11 or qt-mir clients.

This is probably because mir provides scroll motion in rotary ticks - which are usually been translated as 15-times larger than touchpad motion units. The attached patch resolves the problem.
Comment 1 fakey 2016-08-12 14:50:46 UTC
Review of attachment 332799 [details] [review]:

Looks OK, can we leave the -delta_y alone though?

::: gdk/mir/gdkmireventsource.c
@@ +198,3 @@
       event->scroll.direction = GDK_SCROLL_SMOOTH;
+      event->scroll.delta_x = delta_x;
+      event->scroll.delta_y = delta_y;

Yes, for me this fixes the horizontal smooth scrolling, but breaks the vertical smooth scrolling. I guess there's some disparity between the vertical scroll direction in Mir and GTK+? Can we undo the second change here?
Comment 2 Andreas Pokorny 2016-08-12 15:31:08 UTC
(In reply to William Hua from comment #1)
> Review of attachment 332799 [details] [review] [review]:
> 
> Looks OK, can we leave the -delta_y alone though?
> 
> ::: gdk/mir/gdkmireventsource.c
> @@ +198,3 @@
>        event->scroll.direction = GDK_SCROLL_SMOOTH;
> +      event->scroll.delta_x = delta_x;
> +      event->scroll.delta_y = delta_y;
> 
> Yes, for me this fixes the horizontal smooth scrolling, but breaks the
> vertical smooth scrolling. I guess there's some disparity between the
> vertical scroll direction in Mir and GTK+? Can we undo the second change
> here?

I do not know what direction is positive or negative. A user can customize that via server side options. 

We can keep it -delta_y. I just hope this will be in the end common across toolktis.
Comment 3 Andreas Pokorny 2016-08-13 23:32:23 UTC
Created attachment 333250 [details] [review]
[PATCH] Do not filter out small scroll event fractions