_net_active_window shouldn't change desktop, is the general policy concensus.

diff --git a/openbox/event.c b/openbox/event.c
index 7e2766f0d7f9ee1dc2acfd93b06d9e681782dbb4..1b3c1c23e66e86f054ddda8d7f8c0d4cc269ffec 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -1355,7 +1355,7 @@ static void event_handle_client(ObClient *client, XEvent *e)
                 ob_debug_type(OB_DEBUG_APP_BUGS,
                               "_NET_ACTIVE_WINDOW message for window %s is "
                               "missing source indication\n");
-            client_activate(client, FALSE, TRUE, TRUE,
+            client_activate(client, TRUE, TRUE, TRUE,
                             (e->xclient.data.l[0] == 0 ||
                              e->xclient.data.l[0] == 2));
         } else if (msgtype == prop_atoms.net_wm_moveresize) {