Review Request 119001: Add support for high-precision touchpad scrolling

Frank Reininghaus frank78ac at googlemail.com
Sun Jun 29 19:49:26 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119001/#review61207
-----------------------------------------------------------


Thanks for working on this, Kai! I can confirm that it works. This might be a stupid question, but is there a reason to change the code in the "else" branch from "event->delta()" to "event->angleDelta().y()"?

- Frank Reininghaus


On June 28, 2014, 6:47 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119001/
> -----------------------------------------------------------
> 
> (Updated June 28, 2014, 6:47 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> This adds support for QWheelEvent's pixelDelta() which reports pixel-precise scrolling, mainly on touchpads. Since Qt 5.3 this is supported on X11 using XInput2, breaking Dolphin's custom scrolling on those devices.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/kitemviews/kitemlistcontainer.cpp 39e6296 
> 
> Diff: https://git.reviewboard.kde.org/r/119001/diff/
> 
> 
> Testing
> -------
> 
> Tested both vertical and horizontal scrolling using both mouse and touchpad.
> 
> For some reason, although it returns a QPoint with x() and y() values, only y() is ever populated on my machine, even when doing horizontal touchpad gestures. Someone with a horizontally scrollable mousewheel might want to give this a try. I encountered this problem also in QtQuick, so might be a X11 QPA limitation.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20140629/9e4ce22c/attachment.htm>


More information about the kfm-devel mailing list