Cloudef / wlc

High-level Wayland compositor library
MIT License
331 stars 58 forks source link

Touch event reports view by mouse position #266

Closed kozec closed 7 years ago

kozec commented 7 years ago

Hello,

I'm playing with touch events and I've noticed pretty weird behavior - view reported by touch events is not view that user is touching, but view that mouse pointer is above.

Is this intentional?

ddevault commented 7 years ago

The mouse pointer should probably be moved in response to touch events.

kozec commented 7 years ago

The mouse pointer should probably be moved in response to touch events.

Yes, but even doing that is not enough, as first touch event always fires before pointer is moved and thus reports wrong view.

ddevault commented 7 years ago

Sure, it sounds like a bug to me.

kozec commented 7 years ago

By the way, another problem where I'm not 100% sure if it's a bug...

Code in touch.c appears to send touch events with coordinates in screen space, but I'm pretty sure that Wayland client expects them in surface space. Tapping on gnome calculator suggest the same, as button that is actually pressed when I touch its window is offseted and offset depends on window position.

ddevault commented 7 years ago

Definitely sounds like a bug to me.