all repos — openbox @ 0c0ddc36289f39ee32347e775147e7300ce6cf89

openbox fork - make it a bit more like ryudo 

Fix stacking for transients vs helper windows. Fixes bug #3851

This allows transient windows to be above helper windows.  And generally keeps
helper windows below transients, unless they are raised directly.
Dana Jansens danakj@orodu.net
commit

0c0ddc36289f39ee32347e775147e7300ce6cf89

parent

4e0a4fb53b859d818a4184564a441fce29c12be0

1 files changed, 16 insertions(+), 1 deletions(-)

jump to
M openbox/stacking.c → openbox/stacking.c 
@@ -216,6 +216,7 @@ {
     GList *it, *last, *below, *above, *next;
     GList *wins = NULL;
 
+    GList *group_helpers = NULL;
     GList *group_modals = NULL;
     GList *group_trans = NULL;
     GList *modals = NULL;

@@ -246,6 +247,8 @@ ObClient *ch = it->data;
 
                 /* only move windows in the same stacking layer */
                 if (ch->layer == selected->layer &&
+                    /* looking for windows that are transients, and so would
+                       remain above the selected window */
                     client_search_transient(selected, ch))
                 {
                     if (client_is_direct_child(selected, ch)) {

@@ -254,6 +257,13 @@ modals = g_list_prepend(modals, ch);
                         else
                             trans = g_list_prepend(trans, ch);
                     }
+                    else if (client_helper(ch)) {
+                        if (selected->transient) {
+                            /* helpers do not stay above transient windows */
+                            continue;
+                        }
+                        group_helpers = g_list_prepend(group_helpers, ch);
+                    }
                     else {
                         if (ch->modal)
                             group_modals = g_list_prepend(group_modals, ch);

@@ -266,8 +276,13 @@ }
         }
     }
 
-    /* put transients of the selected window right above it */
+    /* put modals above other direct transients */
     wins = g_list_concat(modals, trans);
+
+    /* put helpers below direct transients */
+    wins = g_list_concat(wins, group_helpers);
+
+    /* put the selected window right below these children */
     wins = g_list_append(wins, selected);
 
     /* if selected window is transient for group then raise it above others */