[mutter] pointer: Actually do change the focus during window ops



commit 9fdf487da113ef47809fc9c6b6e43d710f67d3bc
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Wed Nov 26 15:30:04 2014 -0800

    pointer: Actually do change the focus during window ops
    
    Whenever the compositor takes a grab, we're supposed send leave/enter
    events to the current surface, which makes sense, as the compositor
    has stolen the pointer from the client.
    
    I forget why I added the special case in the first place, but it's
    likely a bug that's since been fixed.
    
    This actually fixes a bug: it prevents the need to double-click on
    X11 application titlebars when grabbing them.

 src/wayland/meta-wayland-pointer.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)
---
diff --git a/src/wayland/meta-wayland-pointer.c b/src/wayland/meta-wayland-pointer.c
index 52d13da..faa8e79 100644
--- a/src/wayland/meta-wayland-pointer.c
+++ b/src/wayland/meta-wayland-pointer.c
@@ -73,9 +73,6 @@ sync_focus_surface (MetaWaylandPointer *pointer)
   switch (display->event_route)
     {
     case META_EVENT_ROUTE_WINDOW_OP:
-      /* Don't update the focus surface while we're grabbing a window. */
-      return;
-
     case META_EVENT_ROUTE_COMPOSITOR_GRAB:
       /* The compositor has a grab, so remove our focus... */
       meta_wayland_pointer_set_focus (pointer, NULL);


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