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

["Daniel P. Berrange" <berrange redhat com>]

On Fri, Oct 26, 2007 at 08:48:09AM -0500, Anthony Liguori wrote:
> 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.

Ok, fair enough, lets just do it for mouse events then.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=|