X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Ffocus.c;h=b676c1279956e42ac87b3d015d82c19fec74f8b6;hb=cbbf90a718ecc6836ef7a77b9040aebb9da348b8;hp=a462915740c4e60df713021e5c2f38603eb43fe1;hpb=eee483bb472b0d3e95dd2470aa49656653dd13d6;p=chaz%2Fopenbox

diff --git a/openbox/focus.c b/openbox/focus.c
index a4629157..b676c127 100644
--- a/openbox/focus.c
+++ b/openbox/focus.c
@@ -1,5 +1,4 @@
 #include "event.h"
-#include "config.h"
 #include "openbox.h"
 #include "client.h"
 #include "frame.h"
@@ -16,6 +15,8 @@ GList **focus_order = NULL; /* these lists are created when screen_startup
                                sets the number of desktops */
 
 Window focus_backup = None;
+gboolean focus_new = TRUE;
+gboolean focus_follow = TRUE;
 
 void focus_startup()
 {
@@ -25,11 +26,14 @@ void focus_startup()
     XSetWindowAttributes attrib;
 
     focus_client = NULL;
+    focus_new = TRUE;
+    focus_follow = TRUE;
 
     attrib.override_redirect = TRUE;
     focus_backup = XCreateWindow(ob_display, ob_root,
-				 -100, -100, 1, 1, 0, 0, InputOnly,
-				 CopyFromParent, CWOverrideRedirect, &attrib);
+				 -100, -100, 1, 1, 0,
+                                 CopyFromParent, InputOutput, CopyFromParent,
+                                 CWOverrideRedirect, &attrib);
     XMapRaised(ob_display, focus_backup);
 
     /* start with nothing focused */
@@ -58,8 +62,6 @@ void focus_set_client(Client *client)
     Client *old;
     guint desktop;
 
-    if (client == focus_client) return;
-
     /* uninstall the old colormap, and install the new one */
     screen_install_colormap(focus_client, FALSE);
     screen_install_colormap(client, TRUE);
@@ -68,6 +70,7 @@ void focus_set_client(Client *client)
 	/* when nothing will be focused, send focus to the backup target */
 	XSetInputFocus(ob_display, focus_backup, RevertToPointerRoot,
                        event_lasttime);
+        XSync(ob_display, FALSE);
     }
 
     old = focus_client;
@@ -98,44 +101,47 @@ static gboolean focus_under_pointer()
     guint u;
     GList *it;
 
-    if (XQueryPointer(ob_display, ob_root, &w, &w, &x, &y, &i, &i, &u))
-    {
+    if (XQueryPointer(ob_display, ob_root, &w, &w, &x, &y, &i, &i, &u)) {
         for (it = stacking_list; it != NULL; it = it->next) {
             Client *c = it->data;
             if (c->desktop == screen_desktop &&
                 RECT_CONTAINS(c->frame->area, x, y))
                 break;
         }
-        if (it != NULL) {
-            g_message("fallback (pointer) trying %lx", ((Client*)it->data)->window);
+        if (it != NULL)
             return client_normal(it->data) && client_focus(it->data);
-        }
     }
     return FALSE;
 }
 
 void focus_fallback(gboolean switching_desks)
 {
-    ConfigValue focus_follow;
     GList *it;
-    gboolean fallback = TRUE;
+    gboolean under = FALSE;
+    Client *old = NULL;
+
+    old = focus_client;
 
-    if (!switching_desks) {
-        if (!config_get("focusFollowsMouse", Config_Bool, &focus_follow))
-            g_assert_not_reached();
-        if (focus_follow.bool)
-            fallback = !focus_under_pointer();
+    /* unfocus any focused clients.. they can be focused by Pointer events
+       and such, and then when I try focus them, I won't get a FocusIn event
+       at all for them.
+    */
+    focus_set_client(NULL);
+
+    if (switching_desks) {
+        /* don't skip any windows when switching desktops */
+        old = NULL;
+    } else {
+        if (focus_follow)
+            under = focus_under_pointer();
     }
 
-    if (fallback) {
+    if (!under) {
         for (it = focus_order[screen_desktop]; it != NULL; it = it->next)
-            if (it->data != focus_client && client_normal(it->data)) {
-                g_message("fallback trying %lx", ((Client*)it->data)->window);
+            if (it->data != old && client_normal(it->data))
                 if (client_focus(it->data))
                     break;
-            }
+        if (it == NULL) /* nothing to focus */
+            focus_set_client(NULL);
     }
-
-    if (it == NULL) /* nothing to focus */
-        focus_set_client(NULL);
 }