From: Dana Jansens <danakj@orodu.net>
Date: Sat, 12 May 2007 16:20:17 +0000 (+0000)
Subject: fix for fullscreen things getting lowered when switching desktops
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=4320bf5fc0119f16974d487a5ec5f4d5babdcde3;p=chaz%2Fopenbox

fix for fullscreen things getting lowered when switching desktops
---

diff --git a/openbox/stacking.c b/openbox/stacking.c
index 66aeebcf..0fb19521 100644
--- a/openbox/stacking.c
+++ b/openbox/stacking.c
@@ -426,16 +426,21 @@ void stacking_add_nonintrusive(ObWindow *win)
         }
     }
     if (!it_below) {
-        if (client_search_transient(client, focus_client)) {
-            /* it's focused so put it at the top */
-            stacking_list = g_list_append(stacking_list, win);
-            stacking_raise(win);
-        } else {
-            /* there is no window to put this directly above, so put it at the
-               bottom */
-            stacking_list = g_list_prepend(stacking_list, win);
-            stacking_lower(win);
-        }
+        /* There is no window to put this directly above, so put it at the
+           top, so you know it is there.
+
+           It used to do this only if the window was focused and lower
+           it otherwise.
+
+           We also put it at the top not the bottom to fix a bug with
+           fullscreen windows. When focusLast is off and followsMouse is
+           on, when you switch desktops, the fullscreen window loses
+           focus and goes into its lower layer. If this puts it at the
+           bottom then when you come back to the desktop, the window is
+           at the bottom and won't get focus back.
+        */
+        stacking_list = g_list_append(stacking_list, win);
+        stacking_raise(win);
     } else {
         /* make sure it's not in the wrong layer though ! */
         for (; it_below; it_below = g_list_next(it_below))