Review Request 128821: [frameworks] Implement zooming with a pinch gesture on a touch screen

[Martin Tobias Holmedahl Sandsmark]

> On Oct. 2, 2016, 12:54 p.m., Martin Tobias Holmedahl Sandsmark wrote:
> > I tried to test with pinching on my touchpad, but apparently Qt doesn't handle the gestures from libinput or something.
> 
> Oliver Sander wrote:
>     Touchpad pinching does not work for me, either.  Would you expect that to work with my patch?  Or is some extra code needed?

it needs a patch in qt (I started on it here: http://ix.io/1sta).


- Martin Tobias Holmedahl


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


On Oct. 2, 2016, 6:42 p.m., Oliver Sander wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128821/
> -----------------------------------------------------------
> 
> (Updated Oct. 2, 2016, 6:42 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Bugs: 366781
>     http://bugs.kde.org/show_bug.cgi?id=366781
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> The attached patch implements document zoom controlled by a two-finger pinch gesture on a touch screen.
> 
> The actual zooming was done by copy'n'paste, I don't understand the full details of how zooming works in okular, yet.  So a bit of review would be appreciated.
> 
> 
> Diffs
> -----
> 
>   ui/pageview.h 7c43f51 
>   ui/pageview.cpp 5fbabb2 
> 
> Diff: https://git.reviewboard.kde.org/r/128821/diff/
> 
> 
> Testing
> -------
> 
> I tested it on my Lenovo Thinkpad Yoga, and it works nicely there.
> 
> 
> Thanks,
> 
> Oliver Sander
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20161002/77a6831d/attachment.html>