[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    Re: [PATCH v3 xserver] xwayland-input: Fake crossing to rootwin
From:       Olivier Fourdan <ofourdan () redhat ! com>
Date:       2016-06-23 13:29:21
Message-ID: 1869885522.1167092.1466688561779.JavaMail.zimbra () redhat ! com
[Download RAW message or body]

----- Original Message -----
> On Tue, Jun 21, 2016 at 01:54:35PM +0200, Olivier Fourdan wrote:
> > This partially reverts commit c1565f3.
> > 
> > When the pointer moves from an X11 window to a Wayland native window,
> > no LeaveNotify event is emitted which can lead to various unexpected
> > behaviors like tooltips remaining visible after the pointer has left the
> > window.
> > 
> > Yet the pointer_handle_leave() is called and so is the DIX CheckMotion()
> > but since the pointer enters a Wayland native window with no other
> > Xwayland window matching, DoEnterLeaveEvents() does not get invoked and
> > therefore no LeaveNotify event is sent to the X11 client at the time the
> > pointer leaves the window for a Wayland native surface.
> > 
> > Restore the XYToWindow() handler in xwayland-input that was previously
> > removed with commit c1565f3 and use that handler to pretend that the
> > pointer entered the root window in this case so that the LeaveNotify
> > event is emitted.
> > 
> > Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
> > Acked-by: Daniel Stone <daniels@collabora.com>
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96437
> > ---
> 
>    111a045..7397a21  master -> master
> 
> thanks.

Blimey! just when this gets merged I realize this can confuse focus follow mouse in mutter/gnome-shell...

Sorry about that, I'll post a possible fix shortly, I thionk it's pretty simple to address.

Cheers,
Olivier
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic