[PATCH v3 weston 2/2] Support axis source, axis discrete, frame and axis stop events

Bill Spitzak spitzak at gmail.com
Tue Jan 12 14:41:52 PST 2016


On Mon, Jan 11, 2016 at 4:07 PM, Jonas Ã…dahl <jadahl at gmail.com> wrote:

>
> > > > + widget_pointer_frame_handler_t pointer_frame_handler;
> > > > + widget_axis_source_handler_t axis_source_handler;
> > > > + widget_axis_stop_handler_t axis_stop_handler;
> > > > + widget_axis_discrete_handler_t axis_discrete_handler;
> > >
> > > Should we really make the toytoolkit API this low level? I guess it's
> > > easier this way, so I won't complain further, but it'd be good to have
> > > our toy toolkit try to construct the higher level events the protocol
> > > try to communicate.
> >
> > I think the question is: how much we want the toytoolkit to stay a
> > toy? Composing those events is doable, but how many toytoolkit users will
> > really care about them?
>
> Probably none. It's just a matter of a "toytoolkit should represent how
> the protocol should be interpreted" kind of philosophy. But I understand
> that doing all that extra work just for that reason isn't all that
> reasonable.
>

Another option is to *not* support this in toytoolkit.

Making callbacks like this that don't wait for the frame event is very
misleading, and people are going to copy it (if you think they are going to
read the documentation rather than copy code from toytoolkit, I have a
bridge for sale you may like). I think it would be better to just not
support this at all. Then somebody looking for "how do I get pen tilt from
the tablet" would be forced to read the documentation.

I am not very clear on how this could be done, but it would be nice if
there was a way to do direct-wayland inside toytoolkit. With no support
from toytoolkit, a client should be able to get tablet events. I think all
this requires is the ability to get the wl_surface or xdg_surface proxy
from the toytoolkit window, but maybe I am missing something here...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20160112/60122dc8/attachment.html>


More information about the wayland-devel mailing list