From: Dana Jansens <danakj@orodu.net>
Date: Fri, 28 Mar 2003 10:46:25 +0000 (+0000)
Subject: comment why NotifyAncestor is not ignored
X-Git-Url: https://git.dogcows.com/gitweb?a=commitdiff_plain;h=4d65fe35fffbb1936b28428829153c2b6b58d994;p=chaz%2Fopenbox

comment why NotifyAncestor is not ignored
---

diff --git a/openbox/event.c b/openbox/event.c
index b048e7e6..aaa4c768 100644
--- 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;