comment why NotifyAncestor is not ignored

diff --git a/openbox/event.c b/openbox/event.c
index b048e7e64e7e95babf6c08f0536386d9bb99df5a..aaa4c7684b19f3d3299239be2b8801673608393e 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -223,8 +223,13 @@ void event_process(XEvent *e)
     case FocusIn:
         g_message("FocusIn on %lx mode %d detail %d", window,
                   e->xfocus.mode, e->xfocus.detail);
+        /* NotifyAncestor is not ignored in FocusIn like it is in FocusOut
+           because of RevertToPointerRoot. If the focus ends up reverting to
+           pointer root on a workspace change, then the FocusIn event that we
+           want will be of type NotifyAncestor. This situation does not occur
+           for FocusOut, so it is safely ignored there.
+        */
        if (e->xfocus.detail == NotifyInferior ||
-            /*e->xfocus.detail == NotifyAncestor ||*/
             e->xfocus.detail > NotifyNonlinearVirtual) return;
             g_message("FocusIn on %lx", window);
         break;