Re: [gtk-vnc-devel] [PATCH] Propagate mouse events



Daniel P. Berrange wrote:
On Tue, Oct 23, 2007 at 10:28:35PM -0500, Anthony Liguori wrote:
I needed to add mouse handlers in kvm-test and this seemed to be the only way to get things to work. If anyone knows a better solution, I'm all ears.

Actually I think what your patch is doing is the correct thing. Since we
are a subclass of a generic canvas widget, the only parent for the events
will be the user supplied callbacks. Do we need the same capability for
the keyboard event handlers ?

We can trap the keyboard events just fine so I didn't think it was necessary. I also think we don't want to propagate keyboard events because hopefully that avoids triggering accelerators.

Regards,

Anthony Liguori

Dan.





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]