Don't move windows to current desktop on net_active_window if the event didn't come from the user.

Instead, just activate the flashy thinger.

1 files changed, 6 insertions, 3 deletions

diff --git a/openbox/client.c b/openbox/client.c
index d6321949..39a03468 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -3904,12 +3904,15 @@ static void client_present(ObClient *self, gboolean here, gboolean raise,
     client_focus(self);
 }
 
-/* this function exists to map to the client_activate message in the ewmh,
-   the user arg is unused because nobody uses it correctly anyway. */
+/* this function exists to map to the net_active_window message in the ewmh */
 void client_activate(ObClient *self, gboolean here, gboolean raise,
                      gboolean unshade, gboolean user)
 {
-    client_present(self, here, raise, unshade);
+    if (user || (self->desktop == DESKTOP_ALL ||
+                 self->desktop == screen_desktop))
+        client_present(self, here, raise, unshade);
+    else
+        client_hilite(self, TRUE);
 }
 
 static void client_bring_windows_recursive(ObClient *self,