koma/i3
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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

Browse Source 
This reverts commit 0659a0d98c.

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
This commit is contained in:
Michael Stapelberg
2013-08-27 20:21:11 +02:00
parent 2fe2e005a0
commit 103ab83131
3 changed files with 9 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/x.c
View File

@ -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);
}