Hi Chris,

The newest version of bVNC now targets a newer Android SDK and sports the
Holo UI. So that explains what you're seeing.

I still don't see why you need to call the base method with your patch
applied. Do you have an explanation? It would be best if this is
explainable and if we have an estimation of whether it will break something
for other users.

Thanks!
iordan



On Tue, Mar 18, 2014 at 1:51 PM, Chris Reeves <notifications@github.com>wrote:

> In
> eclipse_projects/bVNC/src/com/iiordanov/bVNC/input/AbstractGestureInputHandler.java:
>
> > @@ -176,9 +176,8 @@ protected boolean handleMouseActions (MotionEvent e) {
> >          final int meta   = e.getMetaState();
> >          final int bstate = e.getButtonState();
> >          RemotePointer p  = canvas.getPointer();
> > -        float scale = canvas.getScale();
> > -        int x = (int)(canvas.getAbsoluteX() +  e.getX()                          / scale);
> > -        int y = (int)(canvas.getAbsoluteY() + (e.getY() - 1.f * canvas.getTop()) / scale);
> > +        int x = getX(e);
>
> This is the issue with simulated-touchpad input. The class that handles it
> overrides getX and getY. Running the base getX and getY code inline, fixes
> the problem.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/iiordanov/remote-desktop-clients/pull/24/files#r10716319>
> .
>



-- 
The conscious mind has only one thread of execution.