all repos — openbox @ dc7a267af0b7e4f1240ea3b203e1b21a7224c179

openbox fork - make it a bit more like ryudo 

there's this case where we DONT want to ignore both events or we end up without anything we know of being focused, even tho something is. this case is now covered. ps i hate focus so much.
Dana Jansens danakj@orodu.net
commit

dc7a267af0b7e4f1240ea3b203e1b21a7224c179

parent

a373b1582e4e40a4f126632a7528553e74fd26f2

1 files changed, 23 insertions(+), 4 deletions(-)

jump to
M openbox/event.c → openbox/event.c 
@@ -341,14 +341,33 @@ #ifdef DEBUG_FOCUS
                     g_message("found pending FocusIn");
 #endif
                     /* is the focused window getting a FocusOut/In back to
-                       itself? */
+                       itself?
+                    */
                     if (fe.xfocus.window == e->xfocus.window &&
                         !event_ignore(&fe, client)) {
+                        /*
+                          if focus_client is not set, then we can't do
+                          this. we need the FocusIn. This happens in the
+                          case when the set_focus_client(NULL) in the
+                          focus_fallback function fires and then
+                          focus_fallback picks the currently focused
+                          window (such as on a SendToDesktop-esque action.
+                        */
+                        if (focus_client) {
 #ifdef DEBUG_FOCUS
-                        g_message("focused window got an Out/In back to "
-                                  "itself IGNORED both");
+                            g_message("focused window got an Out/In back to "
+                                      "itself IGNORED both");
+#endif
+                            return TRUE;
+                        } else {
+                            event_process(&fe);
+#ifdef DEBUG_FOCUS
+                            g_message("focused window got an Out/In back to "
+                                      "itself but focus_client was null "
+                                      "IGNORED just the Out");
 #endif
-                        return TRUE;
+                            return TRUE;
+                        }
                     }
 
                     /* once all the FocusOut's have been dealt with, if there