[PATCH 1/2] cairo-util: Add frame_touch_motion support

[Jason Ekstrand]

On Tue, May 13, 2014 at 5:46 AM, Stanislav Vorobiov
<s.vorobiov at samsung.com>wrote:

> Hi,
>
> Recently I've sent patches to ML that implement just that, see -
> http://lists.freedesktop.org/archives/wayland-devel/2014-May/014602.html
> I got no comments on them though...
>

Yes, these kind of have to be developed side-by-side.  Now would be a good
time for you to test/review/comment on each other's patches.  Now that 1.5
is almost out, we should be able to land more invasive changes.


>
> On 05/13/2014 02:14 PM, Boyan Ding wrote:
> > Hi Jason,
> >
> > In writing v2 of the patches I found out frame_touch_motion
> > isn't that easy as it seems. For there's something tricky.
> >
> > Without frame_touch_motion the code actually works all right - finger
> > down on a button presses the button and finger up releases it,
> > regardless of where the finger has gone.
> >
> > However, if frame_touch_motion is implemented, I think the
> > behavior might be changed to this - finger down presses
> > the button; once the finger leave the button, the press should be
> > cancelled and if the finger enters the region of another button we'll
> > just act as if we "hover" above the button.
>

Yes, that seems perfectly reasonable.


> >
> > If it is implemented like that, we'll have to change the
> > frame_touch structure to add a field that keep track of which
> > button we hover on, just like hover_button in frame_pointer.
> >
> > I'll also add a frame_touch_cancel function, it seems not
> > that hard. Just scan the list of touches and take appropriate
> > actions (cancel button, destroy touch) will do.
>

Yup, that should be easy.

Hope that helps,
--Jason Ekstrand


>
> > I wonder whether my ideas are appropriate.
> >
> > Regards,
> > Boyan Ding
> >
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140513/a9d8c4c1/attachment.html>