handle better the case of choosing the window to focus but it is already focused...

diff --git a/openbox/focus.c b/openbox/focus.c
index 98f81800f4de7092372542d836ed8fa96b07d49a..5c5130f6958a3f1d584ae2f4c3db85f0fe435aed 100644 (file)
--- a/openbox/focus.c
+++ b/openbox/focus.c
@@ -116,9 +116,11 @@ void focus_fallback(gboolean switching_desks)
     ConfigValue focus_follow;
     GList *it;
     gboolean under = FALSE;
+    Client *old = NULL;
 
     if (switching_desks) {
         /* don't skip any windows when switching desktops */
+        old = focus_client;
         focus_client = NULL;
     } else {
         if (!config_get("focusFollowsMouse", Config_Bool, &focus_follow))
@@ -128,10 +130,19 @@ void focus_fallback(gboolean switching_desks)
     }
 
     if (!under) {
-        for (it = focus_order[screen_desktop]; it != NULL; it = it->next)
-            if (it->data != focus_client && client_normal(it->data))
-                if (client_focus(it->data))
+        for (it = focus_order[screen_desktop]; it != NULL; it = it->next) {
+            g_message("fallback trying 0x%lx", ((Client*)it->data)->window);
+            if (it->data != focus_client && client_normal(it->data)) {
+                /* if we're switching desktops, and we get the already focused
+                   window, then we wont get a FocusIn for it, so just restore
+                   the focus_client so that we know it is focused */
+                if (it->data == old) {
+                    focus_client = old;
                     break;
+                } else if (client_focus(it->data))
+                    break;
+            }
+        }
         if (it == NULL) /* nothing to focus */
             focus_set_client(NULL);
     }