Revert "raise fullscreen windows on top of all other X11 windows"

This reverts commit 0659a0d98ca98fcd134c277c6a61ca3c9a04544e.

It was a fix for dzen2 setups, but broke several other use cases.
Because dzen2 is not really important since most people use i3bar
nowadays, let’s revert this.

fixes #1062
fixes #1068
fixes #1070

diff --git a/include/x.h b/include/x.h
index b6bb4a5283702dd2f0110f2122b2d28574f77710..f1fa7e6a7dc3d690bbdb4a5dcc457a5bd568331a 100644 (file)
--- a/include/x.h
+++ b/include/x.h
@@ -93,12 +93,8 @@ void x_push_changes(Con *con);
  * Raises the specified container in the internal stack of X windows. The
  * next call to x_push_changes() will make the change visible in X11.
  *
- * If above_all is true, the X11 window will be raised to the top
- * of the stack. This should only be used for precisely one fullscreen
- * window per output.
- *
  */
-void x_raise_con(Con *con, bool above_all);
+void x_raise_con(Con *con);
 
 /**
  * Sets the WM_NAME property (so, no UTF8, but used only for debugging anyways)

diff --git a/src/render.c b/src/render.c
index 1a7b8fd9ea3dd3aa6d68a2be46485a7febb18c9f..f996d9641d849bc1cd69f0fe6f93825819505aa6 100644 (file)
--- a/src/render.c
+++ b/src/render.c
@@ -70,7 +70,7 @@ static void render_l_output(Con *con) {
     Con *fullscreen = con_get_fullscreen_con(ws, CF_OUTPUT);
     if (fullscreen) {
         fullscreen->rect = con->rect;
-        x_raise_con(fullscreen, true);
+        x_raise_con(fullscreen);
         render_con(fullscreen, true);
         return;
     }
@@ -110,7 +110,7 @@ static void render_l_output(Con *con) {
 
         DLOG("child at (%d, %d) with (%d x %d)\n",
                 child->rect.x, child->rect.y, child->rect.width, child->rect.height);
-        x_raise_con(child, false);
+        x_raise_con(child);
         render_con(child, false);
     }
 }
@@ -208,7 +208,7 @@ void render_con(Con *con, bool render_fullscreen) {
     }
     if (fullscreen) {
         fullscreen->rect = rect;
-        x_raise_con(fullscreen, false);
+        x_raise_con(fullscreen);
         render_con(fullscreen, true);
         return;
     }
@@ -299,7 +299,7 @@ void render_con(Con *con, bool render_fullscreen) {
                 }
                 DLOG("floating child at (%d,%d) with %d x %d\n",
                      child->rect.x, child->rect.y, child->rect.width, child->rect.height);
-                x_raise_con(child, false);
+                x_raise_con(child);
                 render_con(child, false);
             }
         }
@@ -408,7 +408,7 @@ void render_con(Con *con, bool render_fullscreen) {
 
         DLOG("child at (%d, %d) with (%d x %d)\n",
                 child->rect.x, child->rect.y, child->rect.width, child->rect.height);
-        x_raise_con(child, false);
+        x_raise_con(child);
         render_con(child, false);
         i++;
     }
@@ -416,7 +416,7 @@ void render_con(Con *con, bool render_fullscreen) {
     /* in a stacking or tabbed container, we ensure the focused client is raised */
     if (con->layout == L_STACKED || con->layout == L_TABBED) {
         TAILQ_FOREACH_REVERSE(child, &(con->focus_head), focus_head, focused)
-            x_raise_con(child, false);
+            x_raise_con(child);
         if ((child = TAILQ_FIRST(&(con->focus_head)))) {
             /* By rendering the stacked container again, we handle the case
              * that we have a non-leaf-container inside the stack. In that
@@ -430,7 +430,7 @@ void render_con(Con *con, bool render_fullscreen) {
              * top of every stack window. That way, when a new window is opened in
              * the stack, the old window will not obscure part of the decoration
              * (it’s unmapped afterwards). */
-            x_raise_con(con, false);
+            x_raise_con(con);
     }
     }
 }

diff --git a/src/x.c b/src/x.c
index 27d92acad13687499c207414ac0b15f7c2fb99d3..ac5498d2330d466fe03d60b3c7499c127c47356c 100644 (file)
--- a/src/x.c
+++ b/src/x.c
@@ -36,7 +36,6 @@ typedef struct con_state {
     bool mapped;
     bool unmap_now;
     bool child_mapped;
-    bool above_all;
 
     /** The con for which this state is. */
     Con *con;
@@ -900,10 +899,6 @@ void x_push_changes(Con *con) {
 
             xcb_configure_window(conn, prev->id, mask, values);
         }
-        if (state->above_all) {
-            DLOG("above all: 0x%08x\n", state->id);
-            xcb_configure_window(conn, state->id, XCB_CONFIG_WINDOW_STACK_MODE, (uint32_t[]){ XCB_STACK_MODE_ABOVE });
-        }
         state->initial = false;
     }
 
@@ -1029,18 +1024,12 @@ void x_push_changes(Con *con) {
  * Raises the specified container in the internal stack of X windows. The
  * next call to x_push_changes() will make the change visible in X11.
  *
- * If above_all is true, the X11 window will be raised to the top
- * of the stack. This should only be used for precisely one fullscreen
- * window per output.
- *
  */
-void x_raise_con(Con *con, bool above_all) {
+void x_raise_con(Con *con) {
     con_state *state;
     state = state_for_frame(con->frame);
     //DLOG("raising in new stack: %p / %s / %s / xid %08x\n", con, con->name, con->window ? con->window->name_json : "", state->id);
 
-    state->above_all = above_all;
-
     CIRCLEQ_REMOVE(&state_head, state, state);
     CIRCLEQ_INSERT_HEAD(&state_head, state, state);
 }