> 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 > >