Using TabletEvent hiRes positions

Martin Sandsmark martin.sandsmark at kde.org
Sun Aug 7 20:11:12 UTC 2016


Hi!

Sorry for the late reply.


On Thu, Jul 14, 2016 at 10:54:29AM +0200, Oliver Sander wrote:
> @@ -909,7 +909,7 @@ QRect PageViewAnnotator::routeTabletEvent( QTabletEvent * e, PageViewItem * item
>      // figure out the event type and button
>      AnnotatorEngine::decodeEvent( e, &eventType, &button );
> 
> -    const QPointF globalPosF = e->hiResGlobalPos();
> +    const QPointF globalPosF = e->globalPosF();
> [...]
> Note that the patch replaces a hiRes position with a low-res one.  Is this change intentional?
> Because as I understand it, the hiRes position was used for a reason, see

Since Qt5 hiResGlobalPos() and globalPosF() return the same value, if you
look at the code. It is also noted in the documentation and hiResGlobalPos()
is marked as deprecated with a QT_DEPRECATED macro in the header.

-- 
Martin Sandsmark


More information about the Okular-devel mailing list