From: Dana Jansens <danakj@orodu.net>
Date: Tue, 20 Apr 2010 19:20:46 +0000 (-0400)
Subject: don't change stacking stuff on windows when moving them during the unmanage phase.
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=a16449adb643d6416cc71af1bb6b5fde93b95cb0;p=chaz%2Fopenbox

don't change stacking stuff on windows when moving them during the unmanage phase.
---

diff --git a/openbox/client.c b/openbox/client.c
index 605a191f..cc691718 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -3128,11 +3128,16 @@ void client_configure(ObClient *self, gint x, gint y, gint w, gint h,
     /* if it moved between monitors, then this can affect the stacking
        layer of this window or others - for fullscreen windows.
        also if it changed to/from oldschool fullscreen then its layer may
-       change */
-    if (screen_find_monitor(&self->frame->area) !=
-        screen_find_monitor(&oldframe) ||
-        (final && (client_is_oldfullscreen(self, &oldclient) !=
-                   client_is_oldfullscreen(self, &self->area))))
+       change
+
+       watch out tho, don't try change stacking stuff if the window is no
+       longer being managed !
+    */
+    if (self->managed &&
+        (screen_find_monitor(&self->frame->area) !=
+         screen_find_monitor(&oldframe) ||
+         (final && (client_is_oldfullscreen(self, &oldclient) !=
+                    client_is_oldfullscreen(self, &self->area)))))
     {
         client_calc_layer(self);
     }