From 9fdf487da113ef47809fc9c6b6e43d710f67d3bc Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Wed, 26 Nov 2014 15:30:04 -0800 Subject: [PATCH] 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 file changed, 3 deletions(-) diff --git a/src/wayland/meta-wayland-pointer.c b/src/wayland/meta-wayland-pointer.c index 52d13da92..faa8e7902 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);