#include "all.h"
/* Stores the X11 window ID of the currently focused window */
-static xcb_window_t focused_id = XCB_NONE;
+xcb_window_t focused_id = XCB_NONE;
/*
* Describes the X11 state we may modify (map state, position, window stack).
typedef struct con_state {
xcb_window_t id;
bool mapped;
+ bool child_mapped;
+
+ /* For reparenting, we have a flag (need_reparent) and the X ID of the old
+ * frame this window was in. The latter is necessary because we need to
+ * ignore UnmapNotify events (by changing the window event mask). */
+ bool need_reparent;
+ xcb_window_t old_frame;
+
Rect rect;
Rect window_rect;
bool initial;
+ char *name;
+
CIRCLEQ_ENTRY(con_state) state;
CIRCLEQ_ENTRY(con_state) old_state;
} con_state;
/* TODO: better error handling? */
ELOG("No state found\n");
- assert(true);
+ assert(false);
return NULL;
}
values[1] = FRAME_EVENT_MASK;
Rect dims = { -15, -15, 10, 10 };
- con->frame = create_window(conn, dims, XCB_WINDOW_CLASS_INPUT_OUTPUT, -1, false, mask, values);
- con->gc = xcb_generate_id(conn);
- xcb_create_gc(conn, con->gc, con->frame, 0, 0);
+ con->frame = create_window(conn, dims, XCB_WINDOW_CLASS_INPUT_OUTPUT, XCURSOR_CURSOR_POINTER, false, mask, values);
struct con_state *state = scalloc(sizeof(struct con_state));
state->id = con->frame;
state->initial = true;
CIRCLEQ_INSERT_HEAD(&state_head, state, state);
CIRCLEQ_INSERT_HEAD(&old_state_head, state, old_state);
- LOG("adding new state for window id 0x%08x\n", state->id);
+ DLOG("adding new state for window id 0x%08x\n", state->id);
+}
+
+/*
+ * Re-initializes the associated X window state for this container. You have
+ * to call this when you assign a client to an empty container to ensure that
+ * its state gets updated correctly.
+ *
+ */
+void x_reinit(Con *con) {
+ struct con_state *state;
+
+ if ((state = state_for_frame(con->frame)) == NULL) {
+ ELOG("window state not found\n");
+ return;
+ }
+
+ DLOG("resetting state %p to initial\n", state);
+ state->initial = true;
+ state->child_mapped = false;
+ memset(&(state->window_rect), 0, sizeof(Rect));
+}
+
+/*
+ * Reparents the child window of the given container (necessary for sticky
+ * containers). The reparenting happens in the next call of x_push_changes().
+ *
+ */
+void x_reparent_child(Con *con, Con *old) {
+ struct con_state *state;
+ if ((state = state_for_frame(con->frame)) == NULL) {
+ ELOG("window state for con not found\n");
+ return;
+ }
+
+ state->need_reparent = true;
+ state->old_frame = old->frame;
}
+/*
+ * Moves a child window from Container src to Container dest.
+ *
+ */
+void x_move_win(Con *src, Con *dest) {
+ struct con_state *state_src, *state_dest;
+
+ if ((state_src = state_for_frame(src->frame)) == NULL) {
+ ELOG("window state for src not found\n");
+ return;
+ }
+
+ if ((state_dest = state_for_frame(dest->frame)) == NULL) {
+ ELOG("window state for dest not found\n");
+ return;
+ }
+
+ Rect zero = { 0, 0, 0, 0 };
+ if (memcmp(&(state_dest->window_rect), &(zero), sizeof(Rect)) == 0) {
+ memcpy(&(state_dest->window_rect), &(state_src->window_rect), sizeof(Rect));
+ DLOG("COPYING RECT\n");
+ }
+}
+
+/*
+ * Kills the window decoration associated with the given container.
+ *
+ */
void x_con_kill(Con *con) {
con_state *state;
xcb_destroy_window(conn, con->frame);
+ xcb_free_pixmap(conn, con->pixmap);
+ xcb_free_gc(conn, con->pm_gc);
state = state_for_frame(con->frame);
CIRCLEQ_REMOVE(&state_head, state, state);
CIRCLEQ_REMOVE(&old_state_head, state, old_state);
+ FREE(state->name);
+ free(state);
+
+ /* Invalidate focused_id to correctly focus new windows with the same ID */
+ focused_id = XCB_NONE;
}
/*
* Returns true if the client supports the given protocol atom (like WM_DELETE_WINDOW)
*
*/
-static bool window_supports_protocol(xcb_window_t window, xcb_atom_t atom) {
+bool window_supports_protocol(xcb_window_t window, xcb_atom_t atom) {
xcb_get_property_cookie_t cookie;
- xcb_get_wm_protocols_reply_t protocols;
+ xcb_icccm_get_wm_protocols_reply_t protocols;
bool result = false;
- cookie = xcb_get_wm_protocols_unchecked(conn, window, atoms[WM_PROTOCOLS]);
- if (xcb_get_wm_protocols_reply(conn, cookie, &protocols, NULL) != 1)
+ cookie = xcb_icccm_get_wm_protocols_unchecked(conn, window, A_WM_PROTOCOLS);
+ if (xcb_icccm_get_wm_protocols_reply(conn, cookie, &protocols, NULL) != 1)
return false;
/* Check if the client’s protocols have the requested atom set */
if (protocols.atoms[i] == atom)
result = true;
- xcb_get_wm_protocols_reply_wipe(&protocols);
+ xcb_icccm_get_wm_protocols_reply_wipe(&protocols);
return result;
}
-void x_window_kill(xcb_window_t window) {
+/*
+ * Kills the given X11 window using WM_DELETE_WINDOW (if supported).
+ *
+ */
+void x_window_kill(xcb_window_t window, kill_window_t kill_window) {
/* if this window does not support WM_DELETE_WINDOW, we kill it the hard way */
- if (!window_supports_protocol(window, atoms[WM_DELETE_WINDOW])) {
- LOG("Killing window the hard way\n");
- xcb_kill_client(conn, window);
+ if (!window_supports_protocol(window, A_WM_DELETE_WINDOW)) {
+ if (kill_window == KILL_WINDOW) {
+ LOG("Killing specific window 0x%08x\n", window);
+ xcb_destroy_window(conn, window);
+ } else {
+ LOG("Killing the X11 client which owns window 0x%08x\n", window);
+ xcb_kill_client(conn, window);
+ }
return;
}
ev.response_type = XCB_CLIENT_MESSAGE;
ev.window = window;
- ev.type = atoms[WM_PROTOCOLS];
+ ev.type = A_WM_PROTOCOLS;
ev.format = 32;
- ev.data.data32[0] = atoms[WM_DELETE_WINDOW];
+ ev.data.data32[0] = A_WM_DELETE_WINDOW;
ev.data.data32[1] = XCB_CURRENT_TIME;
LOG("Sending WM_DELETE to the client\n");
xcb_flush(conn);
}
+/*
+ * Draws the decoration of the given container onto its parent.
+ *
+ */
void x_draw_decoration(Con *con) {
- Con *parent;
-
- parent = con->parent;
-
- if (con == focused)
- xcb_change_gc_single(conn, parent->gc, XCB_GC_FOREGROUND, get_colorpixel("#FF0000"));
- else xcb_change_gc_single(conn, parent->gc, XCB_GC_FOREGROUND, get_colorpixel("#0C0C0C"));
- xcb_rectangle_t drect = { con->deco_rect.x, con->deco_rect.y, con->deco_rect.width, con->deco_rect.height };
- xcb_poly_fill_rectangle(conn, parent->frame, parent->gc, 1, &drect);
+ Con *parent = con->parent;
+ /* This code needs to run for:
+ * • leaf containers
+ * • non-leaf containers which are in a stacked/tabbed container
+ *
+ * It does not need to run for:
+ * • floating containers (they don’t have a decoration)
+ */
+ if ((!con_is_leaf(con) &&
+ parent->layout != L_STACKED &&
+ parent->layout != L_TABBED) ||
+ con->type == CT_FLOATING_CON)
+ return;
+ DLOG("decoration should be rendered for con %p\n", con);
- if (con->window == NULL) {
+ /* Skip containers whose height is 0 (for example empty dockareas) */
+ if (con->rect.height == 0) {
+ DLOG("height == 0, not rendering\n");
return;
}
- if (con->window->class == NULL) {
- LOG("not rendering decoration, not yet known\n");
+ /* 1: build deco_params and compare with cache */
+ struct deco_render_params *p = scalloc(sizeof(struct deco_render_params));
+
+ /* find out which colors to use */
+ if (con->urgent)
+ p->color = &config.client.urgent;
+ else if (con == focused)
+ p->color = &config.client.focused;
+ else if (con == TAILQ_FIRST(&(parent->focus_head)))
+ p->color = &config.client.focused_inactive;
+ else
+ p->color = &config.client.unfocused;
+
+ p->border_style = con_border_style(con);
+
+ Rect *r = &(con->rect);
+ Rect *w = &(con->window_rect);
+ p->con_rect = (struct width_height){ r->width, r->height };
+ p->con_window_rect = (struct width_height){ w->width, w->height };
+ p->con_deco_rect = con->deco_rect;
+ p->background = config.client.background;
+ p->con_is_leaf = con_is_leaf(con);
+ p->font = config.font.id;
+
+ if (con->deco_render_params != NULL &&
+ (con->window == NULL || !con->window->name_x_changed) &&
+ !parent->pixmap_recreated &&
+ !con->pixmap_recreated &&
+ memcmp(p, con->deco_render_params, sizeof(struct deco_render_params)) == 0) {
+ DLOG("CACHE HIT, not re-rendering\n");
+ free(p);
return;
}
+ DLOG("CACHE MISS\n");
+ Con *next = con;
+ while ((next = TAILQ_NEXT(next, nodes))) {
+ DLOG("Also invalidating cache of %p\n", next);
+ FREE(next->deco_render_params);
+ }
+
+ FREE(con->deco_render_params);
+ con->deco_render_params = p;
+
+ if (con->window != NULL && con->window->name_x_changed)
+ con->window->name_x_changed = false;
+
+ parent->pixmap_recreated = false;
+ con->pixmap_recreated = false;
+
+ /* If the con is in fullscreen mode, the decoration height we work with is set to 0 */
+ Rect deco_rect = con->deco_rect;
+ if (con_get_fullscreen_con(parent) == con)
+ deco_rect.height = 0;
+
+ /* 2: draw the client.background, but only for the parts around the client_rect */
+ if (con->window != NULL) {
+ xcb_rectangle_t background[] = {
+ /* top area */
+ { 0, 0, r->width, w->y },
+ /* bottom area */
+ { 0, (w->y + w->height), r->width, r->height - (w->y + w->height) },
+ /* left area */
+ { 0, 0, w->x, r->height },
+ /* right area */
+ { w->x + w->width, 0, r->width - (w->x + w->width), r->height }
+ };
+#if 0
+ for (int i = 0; i < 4; i++)
+ DLOG("rect is (%d, %d) with %d x %d\n",
+ background[i].x,
+ background[i].y,
+ background[i].width,
+ background[i].height
+ );
+#endif
+
+ xcb_change_gc_single(conn, con->pm_gc, XCB_GC_FOREGROUND, config.client.background);
+ xcb_poly_fill_rectangle(conn, con->pixmap, con->pm_gc, sizeof(background) / sizeof(xcb_rectangle_t), background);
+ }
+
+ /* 3: draw a rectangle in border color around the client */
+ if (p->border_style != BS_NONE && p->con_is_leaf) {
+ Rect br = con_border_style_rect(con);
+#if 0
+ DLOG("con->rect spans %d x %d\n", con->rect.width, con->rect.height);
+ DLOG("border_rect spans (%d, %d) with %d x %d\n", br.x, br.y, br.width, br.height);
+ DLOG("window_rect spans (%d, %d) with %d x %d\n", con->window_rect.x, con->window_rect.y, con->window_rect.width, con->window_rect.height);
+#endif
+
+ /* These rectangles represents the border around the child window
+ * (left, bottom and right part). We don’t just fill the whole
+ * rectangle because some childs are not freely resizable and we want
+ * their background color to "shine through". */
+ xcb_change_gc_single(conn, con->pm_gc, XCB_GC_FOREGROUND, p->color->background);
+ xcb_rectangle_t borders[] = {
+ { 0, 0, br.x, r->height },
+ { 0, r->height + br.height + br.y, r->width, r->height },
+ { r->width + br.width + br.x, 0, r->width, r->height }
+ };
+ xcb_poly_fill_rectangle(conn, con->pixmap, con->pm_gc, 3, borders);
+ /* 1pixel border needs an additional line at the top */
+ if (p->border_style == BS_1PIXEL) {
+ xcb_rectangle_t topline = { br.x, 0, con->rect.width + br.width + br.x, br.y };
+ xcb_poly_fill_rectangle(conn, con->pixmap, con->pm_gc, 1, &topline);
+ }
+ }
+
+ /* if this is a borderless/1pixel window, we don’t * need to render the
+ * decoration. */
+ if (p->border_style != BS_NORMAL) {
+ DLOG("border style not BS_NORMAL, aborting rendering of decoration\n");
+ goto update_pixmaps;
+ }
+
+ /* 4: paint the bar */
+ xcb_change_gc_single(conn, parent->pm_gc, XCB_GC_FOREGROUND, p->color->background);
+ xcb_rectangle_t drect = { con->deco_rect.x, con->deco_rect.y, con->deco_rect.width, con->deco_rect.height };
+ xcb_poly_fill_rectangle(conn, parent->pixmap, parent->pm_gc, 1, &drect);
+
+ /* 5: draw the two lines in border color */
+ xcb_draw_line(conn, parent->pixmap, parent->pm_gc, p->color->border,
+ con->deco_rect.x, /* x */
+ con->deco_rect.y, /* y */
+ con->deco_rect.x + con->deco_rect.width, /* to_x */
+ con->deco_rect.y); /* to_y */
+ xcb_draw_line(conn, parent->pixmap, parent->pm_gc, p->color->border,
+ con->deco_rect.x, /* x */
+ con->deco_rect.y + con->deco_rect.height - 1, /* y */
+ con->deco_rect.x + con->deco_rect.width, /* to_x */
+ con->deco_rect.y + con->deco_rect.height - 1); /* to_y */
+
+ /* 6: draw the title */
+ uint32_t mask = XCB_GC_FOREGROUND | XCB_GC_BACKGROUND | XCB_GC_FONT;
+ uint32_t values[] = { p->color->text, p->color->background, config.font.id };
+ xcb_change_gc(conn, parent->pm_gc, mask, values);
+ int text_offset_y = config.font.height + (con->deco_rect.height - config.font.height) / 2 - 1;
+
+ struct Window *win = con->window;
+ if (win == NULL || win->name_x == NULL) {
+ /* this is a non-leaf container, we need to make up a good description */
+ // TODO: use a good description instead of just "another container"
+ xcb_image_text_8(
+ conn,
+ strlen("another container"),
+ parent->pixmap,
+ parent->pm_gc,
+ con->deco_rect.x + 2,
+ con->deco_rect.y + text_offset_y,
+ "another container"
+ );
+
+ goto update_pixmaps;
+ }
+
+ int indent_level = 0,
+ indent_mult = 0;
+ Con *il_parent = parent;
+ if (il_parent->layout != L_STACKED) {
+ while (1) {
+ DLOG("il_parent = %p, layout = %d\n", il_parent, il_parent->layout);
+ if (il_parent->layout == L_STACKED)
+ indent_level++;
+ if (il_parent->type == CT_WORKSPACE || il_parent->type == CT_DOCKAREA || il_parent->type == CT_OUTPUT)
+ break;
+ il_parent = il_parent->parent;
+ indent_mult++;
+ }
+ }
+ DLOG("indent_level = %d, indent_mult = %d\n", indent_level, indent_mult);
+ int indent_px = (indent_level * 5) * indent_mult;
+
+ if (win->uses_net_wm_name)
+ xcb_image_text_16(
+ conn,
+ win->name_len,
+ parent->pixmap,
+ parent->pm_gc,
+ con->deco_rect.x + 2 + indent_px,
+ con->deco_rect.y + text_offset_y,
+ (xcb_char2b_t*)win->name_x
+ );
+ else
+ xcb_image_text_8(
+ conn,
+ win->name_len,
+ parent->pixmap,
+ parent->pm_gc,
+ con->deco_rect.x + 2 + indent_px,
+ con->deco_rect.y + text_offset_y,
+ win->name_x
+ );
+
+update_pixmaps:
+ xcb_clear_area(conn, false, con->frame, 0, 0, con->rect.width, con->rect.height);
+ xcb_clear_area(conn, false, parent->frame, 0, 0, parent->rect.width, parent->rect.height);
+}
+
+/*
+ * Recursively calls x_draw_decoration. This cannot be done in x_push_node
+ * because x_push_node uses focus order to recurse (see the comment above)
+ * while drawing the decoration needs to happen in the actual order.
+ *
+ */
+static void x_deco_recurse(Con *con) {
+ Con *current;
+
+ TAILQ_FOREACH(current, &(con->nodes_head), nodes)
+ x_deco_recurse(current);
- LOG("should render text %s onto %p / %s\n", con->window->class, parent, parent->name);
+ TAILQ_FOREACH(current, &(con->floating_head), floating_windows)
+ x_deco_recurse(current);
- xcb_change_gc_single(conn, parent->gc, XCB_GC_FOREGROUND, get_colorpixel("#FFFFFF"));
- xcb_image_text_8(
- conn,
- strlen(con->window->class),
- parent->frame,
- parent->gc,
- con->deco_rect.x,
- con->deco_rect.y + 14,
- con->window->class
- );
+ if ((con->type != CT_ROOT && con->type != CT_OUTPUT) &&
+ con->mapped)
+ x_draw_decoration(con);
}
/*
* It recursively traverses all children of the given node.
*
*/
-static void x_push_node(Con *con) {
+void x_push_node(Con *con) {
Con *current;
con_state *state;
+ Rect rect = con->rect;
- LOG("Pushing changes for node %p / %s\n", con, con->name);
+ //DLOG("Pushing changes for node %p / %s\n", con, con->name);
state = state_for_frame(con->frame);
- /* map/unmap if map state changed */
- if (state->mapped != con->mapped) {
- if (!con->mapped) {
- LOG("unmapping container\n");
- xcb_unmap_window(conn, con->frame);
- } else {
- if (state->initial && con->window != NULL) {
- LOG("mapping child window\n");
- xcb_map_window(conn, con->window->id);
+ if (state->name != NULL) {
+ DLOG("pushing name %s for con %p\n", state->name, con);
+
+ xcb_change_property(conn, XCB_PROP_MODE_REPLACE, con->frame,
+ A_WM_NAME, A_STRING, 8, strlen(state->name), state->name);
+ FREE(state->name);
+ }
+
+ if (con->window == NULL) {
+ /* Calculate the height of all window decorations which will be drawn on to
+ * this frame. */
+ uint32_t max_y = 0, max_height = 0;
+ TAILQ_FOREACH(current, &(con->nodes_head), nodes) {
+ Rect *dr = &(current->deco_rect);
+ if (dr->y >= max_y && dr->height >= max_height) {
+ max_y = dr->y;
+ max_height = dr->height;
}
- LOG("mapping container\n");
- xcb_map_window(conn, con->frame);
}
- state->mapped = con->mapped;
+ rect.height = max_y + max_height;
+ if (rect.height == 0) {
+ DLOG("Unmapping container %p because it does not contain anything.\n", con);
+ con->mapped = false;
+ }
}
+ /* reparent the child window (when the window was moved due to a sticky
+ * container) */
+ if (state->need_reparent && con->window != NULL) {
+ DLOG("Reparenting child window\n");
+
+ /* Temporarily set the event masks to XCB_NONE so that we won’t get
+ * UnmapNotify events (otherwise the handler would close the container).
+ * These events are generated automatically when reparenting. */
+ uint32_t values[] = { XCB_NONE };
+ xcb_change_window_attributes(conn, state->old_frame, XCB_CW_EVENT_MASK, values);
+ xcb_change_window_attributes(conn, con->window->id, XCB_CW_EVENT_MASK, values);
+
+ xcb_reparent_window(conn, con->window->id, con->frame, 0, 0);
+
+ values[0] = FRAME_EVENT_MASK;
+ xcb_change_window_attributes(conn, state->old_frame, XCB_CW_EVENT_MASK, values);
+ values[0] = CHILD_EVENT_MASK;
+ xcb_change_window_attributes(conn, con->window->id, XCB_CW_EVENT_MASK, values);
+
+ state->old_frame = XCB_NONE;
+ state->need_reparent = false;
+
+ con->ignore_unmap++;
+ DLOG("ignore_unmap for reparenting of con %p (win 0x%08x) is now %d\n",
+ con, con->window->id, con->ignore_unmap);
+ }
+
+ bool fake_notify = false;
/* set new position if rect changed */
- if (memcmp(&(state->rect), &(con->rect), sizeof(Rect)) != 0) {
- LOG("setting rect (%d, %d, %d, %d)\n", con->rect.x, con->rect.y, con->rect.width, con->rect.height);
- xcb_set_window_rect(conn, con->frame, con->rect);
- memcpy(&(state->rect), &(con->rect), sizeof(Rect));
+ if (memcmp(&(state->rect), &rect, sizeof(Rect)) != 0) {
+ /* We first create the new pixmap, then render to it, set it as the
+ * background and only afterwards change the window size. This reduces
+ * flickering. */
+
+ /* As the pixmap only depends on the size and not on the position, it
+ * is enough to check if width/height have changed. Also, we don’t
+ * create a pixmap at all when the window is actually not visible
+ * (height == 0). */
+ if (rect.height > 0 &&
+ (state->rect.width != rect.width ||
+ state->rect.height != rect.height)) {
+ DLOG("CACHE: creating new pixmap for con %p (old: %d x %d, new: %d x %d)\n",
+ con, state->rect.width, state->rect.height,
+ rect.width, rect.height);
+ if (con->pixmap == 0) {
+ con->pixmap = xcb_generate_id(conn);
+ con->pm_gc = xcb_generate_id(conn);
+ } else {
+ xcb_free_pixmap(conn, con->pixmap);
+ xcb_free_gc(conn, con->pm_gc);
+ }
+ xcb_create_pixmap(conn, root_depth, con->pixmap, con->frame, rect.width, rect.height);
+ xcb_create_gc(conn, con->pm_gc, con->pixmap, 0, 0);
+
+ con->pixmap_recreated = true;
+
+ /* Render the decoration now to make the correct decoration visible
+ * from the very first moment. Later calls will be cached, so this
+ * doesn’t hurt performance. */
+ x_deco_recurse(con);
+
+ uint32_t values[] = { con->pixmap };
+ xcb_change_window_attributes(conn, con->frame, XCB_CW_BACK_PIXMAP, values);
+ }
+
+ DLOG("setting rect (%d, %d, %d, %d)\n", rect.x, rect.y, rect.width, rect.height);
+ xcb_set_window_rect(conn, con->frame, rect);
+
+ memcpy(&(state->rect), &rect, sizeof(Rect));
+ fake_notify = true;
}
/* dito, but for child windows */
- if (memcmp(&(state->window_rect), &(con->window_rect), sizeof(Rect)) != 0) {
- LOG("setting window rect (%d, %d, %d, %d)\n",
+ if (con->window != NULL &&
+ memcmp(&(state->window_rect), &(con->window_rect), sizeof(Rect)) != 0) {
+ DLOG("setting window rect (%d, %d, %d, %d)\n",
con->window_rect.x, con->window_rect.y, con->window_rect.width, con->window_rect.height);
xcb_set_window_rect(conn, con->window->id, con->window_rect);
- memcpy(&(state->rect), &(con->rect), sizeof(Rect));
+ memcpy(&(state->window_rect), &(con->window_rect), sizeof(Rect));
+ fake_notify = true;
+ }
+
+ /* Map if map state changed, also ensure that the child window
+ * is changed if we are mapped *and* in initial state (meaning the
+ * container was empty before, but now got a child). Unmaps are handled in
+ * x_push_node_unmaps(). */
+ if ((state->mapped != con->mapped || (con->mapped && state->initial)) &&
+ con->mapped) {
+ xcb_void_cookie_t cookie;
+
+ if (con->window != NULL) {
+ /* Set WM_STATE_NORMAL because GTK applications don’t want to
+ * drag & drop if we don’t. Also, xprop(1) needs it. */
+ long data[] = { XCB_ICCCM_WM_STATE_NORMAL, XCB_NONE };
+ xcb_change_property(conn, XCB_PROP_MODE_REPLACE, con->window->id,
+ A_WM_STATE, A_WM_STATE, 32, 2, data);
+ }
+
+ if (!state->child_mapped && con->window != NULL) {
+ cookie = xcb_map_window(conn, con->window->id);
+ DLOG("mapping child window (serial %d)\n", cookie.sequence);
+ /* Ignore enter_notifies which are generated when mapping */
+ add_ignore_event(cookie.sequence, 0);
+ state->child_mapped = true;
+ }
+
+ cookie = xcb_map_window(conn, con->frame);
+ DLOG("mapping container (serial %d)\n", cookie.sequence);
+ /* Ignore enter_notifies which are generated when mapping */
+ add_ignore_event(cookie.sequence, 0);
+ state->mapped = con->mapped;
+ }
+
+ if (fake_notify) {
+ DLOG("Sending fake configure notify\n");
+ fake_absolute_configure_notify(con);
+ }
+
+ /* Handle all children and floating windows of this node. We recurse
+ * in focus order to display the focused client in a stack first when
+ * switching workspaces (reduces flickering). */
+ TAILQ_FOREACH(current, &(con->focus_head), focused)
+ x_push_node(current);
+}
+
+/*
+ * Same idea as in x_push_node(), but this function only unmaps windows. It is
+ * necessary to split this up to handle new fullscreen clients properly: The
+ * new window needs to be mapped and focus needs to be set *before* the
+ * underlying windows are unmapped. Otherwise, focus will revert to the
+ * PointerRoot and will then be set to the new window, generating unnecessary
+ * FocusIn/FocusOut events.
+ *
+ */
+static void x_push_node_unmaps(Con *con) {
+ Con *current;
+ con_state *state;
+
+ //DLOG("Pushing changes (with unmaps) for node %p / %s\n", con, con->name);
+ state = state_for_frame(con->frame);
+
+ /* map/unmap if map state changed, also ensure that the child window
+ * is changed if we are mapped *and* in initial state (meaning the
+ * container was empty before, but now got a child) */
+ if ((state->mapped != con->mapped || (con->mapped && state->initial)) &&
+ !con->mapped) {
+ xcb_void_cookie_t cookie;
+ if (con->window != NULL) {
+ /* Set WM_STATE_WITHDRAWN, it seems like Java apps need it */
+ long data[] = { XCB_ICCCM_WM_STATE_WITHDRAWN, XCB_NONE };
+ xcb_change_property(conn, XCB_PROP_MODE_REPLACE, con->window->id,
+ A_WM_STATE, A_WM_STATE, 32, 2, data);
+ }
+
+ cookie = xcb_unmap_window(conn, con->frame);
+ DLOG("unmapping container (serial %d)\n", cookie.sequence);
+ /* we need to increase ignore_unmap for this container (if it
+ * contains a window) and for every window "under" this one which
+ * contains a window */
+ if (con->window != NULL) {
+ con->ignore_unmap++;
+ DLOG("ignore_unmap for con %p (frame 0x%08x) now %d\n", con, con->frame, con->ignore_unmap);
+ }
+ /* Ignore enter_notifies which are generated when unmapping */
+ add_ignore_event(cookie.sequence, 0);
+ state->mapped = con->mapped;
}
/* handle all children and floating windows of this node */
TAILQ_FOREACH(current, &(con->nodes_head), nodes)
- x_push_node(current);
+ x_push_node_unmaps(current);
TAILQ_FOREACH(current, &(con->floating_head), floating_windows)
- x_push_node(current);
-
- if (con->type != CT_ROOT && con->type != CT_OUTPUT)
- x_draw_decoration(con);
+ x_push_node_unmaps(current);
}
/*
* Pushes all changes (state of each node, see x_push_node() and the window
* stack) to X11.
*
+ * NOTE: We need to push the stack first so that the windows have the correct
+ * stacking order. This is relevant for workspace switching where we map the
+ * windows because mapping may generate EnterNotify events. When they are
+ * generated in the wrong order, this will cause focus problems when switching
+ * workspaces.
+ *
*/
void x_push_changes(Con *con) {
con_state *state;
- LOG("\n\n PUSHING CHANGES\n\n");
- x_push_node(con);
-
- LOG("-- PUSHING FOCUS STACK --\n");
+ DLOG("-- PUSHING WINDOW STACK --\n");
+ //DLOG("Disabling EnterNotify\n");
+ uint32_t values[1] = { XCB_NONE };
+ CIRCLEQ_FOREACH_REVERSE(state, &state_head, state) {
+ xcb_change_window_attributes(conn, state->id, XCB_CW_EVENT_MASK, values);
+ }
+ //DLOG("Done, EnterNotify disabled\n");
+ bool order_changed = false;
/* X11 correctly represents the stack if we push it from bottom to top */
CIRCLEQ_FOREACH_REVERSE(state, &state_head, state) {
- LOG("stack: 0x%08x\n", state->id);
+ //DLOG("stack: 0x%08x\n", state->id);
con_state *prev = CIRCLEQ_PREV(state, state);
con_state *old_prev = CIRCLEQ_PREV(state, old_state);
- if ((state->initial || prev != old_prev) && prev != CIRCLEQ_END(&state_head)) {
- state->initial = false;
- LOG("Stacking 0x%08x above 0x%08x\n", prev->id, state->id);
+ if (prev != old_prev)
+ order_changed = true;
+ if ((state->initial || order_changed) && prev != CIRCLEQ_END(&state_head)) {
+ DLOG("Stacking 0x%08x above 0x%08x\n", prev->id, state->id);
uint32_t mask = 0;
mask |= XCB_CONFIG_WINDOW_SIBLING;
mask |= XCB_CONFIG_WINDOW_STACK_MODE;
xcb_configure_window(conn, prev->id, mask, values);
}
+ state->initial = false;
+ }
+ //DLOG("Re-enabling EnterNotify\n");
+ values[0] = FRAME_EVENT_MASK;
+ CIRCLEQ_FOREACH_REVERSE(state, &state_head, state) {
+ xcb_change_window_attributes(conn, state->id, XCB_CW_EVENT_MASK, values);
}
+ //DLOG("Done, EnterNotify re-enabled\n");
+
+ DLOG("\n\n PUSHING CHANGES\n\n");
+ x_push_node(con);
+ x_deco_recurse(con);
xcb_window_t to_focus = focused->frame;
if (focused->window != NULL)
to_focus = focused->window->id;
+ DLOG("focused_id = 0x%08x, to_focus = 0x%08x\n", focused_id, to_focus);
if (focused_id != to_focus) {
- LOG("Updating focus\n");
- xcb_set_input_focus(conn, XCB_INPUT_FOCUS_POINTER_ROOT, to_focus, XCB_CURRENT_TIME);
+ if (!focused->mapped) {
+ DLOG("Not updating focus (to %p / %s), focused window is not mapped.\n", focused, focused->name);
+ /* Invalidate focused_id to correctly focus new windows with the same ID */
+ focused_id = XCB_NONE;
+ } else {
+ DLOG("Updating focus (focused: %p / %s)\n", focused, focused->name);
+ /* We remove XCB_EVENT_MASK_FOCUS_CHANGE from the event mask to get
+ * no focus change events for our own focus changes. We only want
+ * these generated by the clients. */
+ if (focused->window != NULL) {
+ values[0] = CHILD_EVENT_MASK & ~(XCB_EVENT_MASK_FOCUS_CHANGE);
+ xcb_change_window_attributes(conn, focused->window->id, XCB_CW_EVENT_MASK, values);
+ }
+ xcb_set_input_focus(conn, XCB_INPUT_FOCUS_POINTER_ROOT, to_focus, XCB_CURRENT_TIME);
+ if (focused->window != NULL) {
+ values[0] = CHILD_EVENT_MASK;
+ xcb_change_window_attributes(conn, focused->window->id, XCB_CW_EVENT_MASK, values);
+ }
+
+ if (focused->window != NULL &&
+ focused->window->needs_take_focus) {
+ send_take_focus(to_focus);
+ }
+
+ ewmh_update_active_window(to_focus);
+ focused_id = to_focus;
+ }
+ }
+
+ if (focused_id == XCB_NONE) {
+ DLOG("Still no window focused, better set focus to the root window\n");
+ xcb_set_input_focus(conn, XCB_INPUT_FOCUS_POINTER_ROOT, root, XCB_CURRENT_TIME);
+ focused_id = root;
}
xcb_flush(conn);
- LOG("\n\n ENDING CHANGES\n\n");
+ DLOG("\n\n ENDING CHANGES\n\n");
+
+ x_push_node_unmaps(con);
/* save the current stack as old stack */
CIRCLEQ_FOREACH(state, &state_head, state) {
CIRCLEQ_REMOVE(&old_state_head, state, old_state);
CIRCLEQ_INSERT_TAIL(&old_state_head, state, old_state);
}
- CIRCLEQ_FOREACH(state, &old_state_head, old_state) {
- LOG("old stack: 0x%08x\n", state->id);
- }
+ //CIRCLEQ_FOREACH(state, &old_state_head, old_state) {
+ // DLOG("old stack: 0x%08x\n", state->id);
+ //}
+
+ xcb_flush(conn);
}
/*
*/
void x_raise_con(Con *con) {
con_state *state;
- LOG("raising in new stack: %p / %s\n", con, con->name);
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);
- LOG("found state entry, moving to top\n");
CIRCLEQ_REMOVE(&state_head, state, state);
CIRCLEQ_INSERT_HEAD(&state_head, state, state);
}
+
+/*
+ * Sets the WM_NAME property (so, no UTF8, but used only for debugging anyways)
+ * of the given name. Used for properly tagging the windows for easily spotting
+ * i3 windows in xwininfo -root -all.
+ *
+ */
+void x_set_name(Con *con, const char *name) {
+ struct con_state *state;
+
+ if ((state = state_for_frame(con->frame)) == NULL) {
+ ELOG("window state not found\n");
+ return;
+ }
+
+ FREE(state->name);
+ state->name = sstrdup(name);
+}
+
+/*
+ * Sets up i3 specific atoms (I3_SOCKET_PATH and I3_CONFIG_PATH)
+ *
+ */
+void x_set_i3_atoms() {
+ xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A_I3_SOCKET_PATH, A_UTF8_STRING, 8,
+ (current_socketpath == NULL ? 0 : strlen(current_socketpath)),
+ current_socketpath);
+ xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A_I3_CONFIG_PATH, A_UTF8_STRING, 8,
+ strlen(current_configpath), current_configpath);
+}
projects / i3 / i3 / blobdiff