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

Instead, just activate the flashy thinger.

diff --git a/openbox/client.c b/openbox/client.c
index d6321949a1dee2a0f1c2912c604d269550c2d295..39a034683876823f7d76a98346f008d0db106a36 100644 (file)
--- 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,