X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fstacking.c;h=1b3833b40d700a5fc9ad7ade517ebb65ae672e06;hb=c4ec27c27534d19543324765330db65596791e26;hp=3a556fc4abb24f71a78efb3a856f366a0b798b8e;hpb=122d55fbadea0409fbc902a1740e1c8ff3aecd88;p=chaz%2Fopenbox

diff --git a/openbox/stacking.c b/openbox/stacking.c
index 3a556fc4..1b3833b4 100644
--- a/openbox/stacking.c
+++ b/openbox/stacking.c
@@ -1,11 +1,11 @@
 #include "openbox.h"
 #include "prop.h"
+#include "screen.h"
 #include "focus.h"
 #include "client.h"
 #include "group.h"
 #include "frame.h"
 #include "window.h"
-#include <glib.h>
 
 GList  *stacking_list = NULL;
 
@@ -18,7 +18,7 @@ void stacking_set_list()
     /* on shutdown, don't update the properties, so that we can read it back
        in on startup and re-stack the windows as they were before we shut down
     */
-    if (ob_state == OB_STATE_EXITING) return;
+    if (ob_state() == OB_STATE_EXITING) return;
 
     /* create an array of the window ids (from bottom to top,
        reverse order!) */
@@ -30,8 +30,8 @@ void stacking_set_list()
         }
     }
 
-    PROP_SETA32(ob_root, net_client_list_stacking, window,
-                (guint32*)windows, i);
+    PROP_SETA32(RootWindow(ob_display, ob_screen),
+                net_client_list_stacking, window, (guint32*)windows, i);
 
     g_free(windows);
 }
@@ -53,7 +53,7 @@ static void do_restack(GList *wins, GList *before)
     win = g_new(Window, g_list_length(wins) + 1);
 
     if (before == stacking_list)
-        win[0] = focus_backup;
+        win[0] = screen_support_win;
     else if (!before)
         win[0] = window_top(g_list_last(stacking_list)->data);
     else
@@ -141,11 +141,14 @@ static GList *pick_windows(ObClient *top, ObClient *selected, gboolean raise)
     int i, n;
     GList *modals = NULL;
     GList *trans = NULL;
-    GList *modal_sel_it = NULL; /* the selected guy if modal */
-    GList *trans_sel_it = NULL; /* the selected guy if not */
+    GList *modal_sel = NULL; /* the selected guys if modal */
+    GList *trans_sel = NULL; /* the selected guys if not */
 
     /* remove first so we can't run into ourself */
-    stacking_list = g_list_remove(stacking_list, top);
+    if ((it = g_list_find(stacking_list, top)))
+        stacking_list = g_list_delete_link(stacking_list, it);
+    else
+        return NULL;
 
     i = 0;
     n = g_slist_length(top->transients);
@@ -155,26 +158,32 @@ static GList *pick_windows(ObClient *top, ObClient *selected, gboolean raise)
 
         if ((sit = g_slist_find(top->transients, it->data))) {
             ObClient *c = sit->data;
+            gboolean sel_child;
 
             ++i;
 
+            if (c == selected)
+                sel_child = TRUE;
+            else
+                sel_child = client_search_transient(c, selected) != NULL;
+
             if (!c->modal) {
-                if (c != selected) {
+                if (!sel_child) {
                     trans = g_list_concat(trans,
-                                           pick_windows(c, selected, raise));
+                                          pick_windows(c, selected, raise));
                 } else {
-                    g_assert(modal_sel_it == NULL);
-                    g_assert(trans_sel_it == NULL);
-                    trans_sel_it = pick_windows(c, selected, raise);
+                    trans_sel = g_list_concat(trans_sel,
+                                                 pick_windows(c, selected,
+                                                              raise));
                 }
             } else {
-                if (c != selected) {
+                if (!sel_child) {
                     modals = g_list_concat(modals,
                                            pick_windows(c, selected, raise));
                 } else {
-                    g_assert(modal_sel_it == NULL);
-                    g_assert(trans_sel_it == NULL);
-                    modal_sel_it = pick_windows(c, selected, raise);
+                    modal_sel = g_list_concat(modal_sel,
+                                                 pick_windows(c, selected,
+                                                              raise));
                 }
             }
             /* if we dont have a prev then start back at the beginning,
@@ -183,11 +192,11 @@ static GList *pick_windows(ObClient *top, ObClient *selected, gboolean raise)
         }
     }
 
-    ret = g_list_concat((raise ? modal_sel_it : modals),
-                        (raise ? modals : modal_sel_it));
+    ret = g_list_concat((raise ? modal_sel : modals),
+                        (raise ? modals : modal_sel));
 
-    ret = g_list_concat(ret, (raise ? trans_sel_it : trans));
-    ret = g_list_concat(ret, (raise ? trans : trans_sel_it));
+    ret = g_list_concat(ret, (raise ? trans_sel : trans));
+    ret = g_list_concat(ret, (raise ? trans : trans_sel));
 
 
     /* add itself */
@@ -196,7 +205,8 @@ static GList *pick_windows(ObClient *top, ObClient *selected, gboolean raise)
     return ret;
 }
 
-static GList *pick_group_windows(ObClient *top, ObClient *selected, gboolean raise)
+static GList *pick_group_windows(ObClient *top, ObClient *selected,
+                                 gboolean raise)
 {
     GList *ret = NULL;
     GList *it, *next, *prev;
@@ -234,7 +244,7 @@ void stacking_raise(ObWindow *window)
         selected = WINDOW_AS_CLIENT(window);
         c = client_search_top_transient(selected);
         wins = pick_windows(c, selected, TRUE);
-        wins = g_list_concat(wins, pick_group_windows(c, selected, TRUE));
+        /*wins = g_list_concat(wins, pick_group_windows(c, selected, TRUE));*/
     } else {
         wins = g_list_append(NULL, window);
         stacking_list = g_list_remove(stacking_list, window);
@@ -253,7 +263,7 @@ void stacking_lower(ObWindow *window)
         selected = WINDOW_AS_CLIENT(window);
         c = client_search_top_transient(selected);
         wins = pick_windows(c, selected, FALSE);
-        wins = g_list_concat(pick_group_windows(c, selected, FALSE), wins);
+        /*wins = g_list_concat(pick_group_windows(c, selected, FALSE), wins);*/
     } else {
         wins = g_list_append(NULL, window);
         stacking_list = g_list_remove(stacking_list, window);
@@ -267,7 +277,7 @@ void stacking_add(ObWindow *win)
     ObStackingLayer l;
     GList *wins;
 
-    g_assert(focus_backup != None); /* make sure I dont break this in the
+    g_assert(screen_support_win != None); /* make sure I dont break this in the
                                        future */
 
     l = window_layer(win);