X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;ds=sidebyside;f=src%2Fmanage.c;h=d84ba1b24d328e90ae367af3babe24ff6ea48f4f;hb=22be7bc986f51e8dc798fcd5f6bb72e3ed87ba0c;hp=22c2814fa4fcdcfb10f697e388b82a9640988d4b;hpb=819980ce2b2e3549eb99d91557dfe0d0602fa540;p=i3%2Fi3 diff --git a/src/manage.c b/src/manage.c index 22c2814f..d84ba1b2 100644 --- a/src/manage.c +++ b/src/manage.c @@ -1,15 +1,18 @@ +#undef I3__FILE__ +#define I3__FILE__ "manage.c" /* * vim:ts=4:sw=4:expandtab * * i3 - an improved dynamic tiling window manager - * © 2009-2011 Michael Stapelberg and contributors (see also: LICENSE) + * © 2009-2013 Michael Stapelberg and contributors (see also: LICENSE) * - * manage.c: Contains all functions for initially managing new windows - * (or existing ones on restart). + * manage.c: Initially managing new windows (or existing ones on restart). * */ - #include "all.h" +#include "yajl_utils.h" + +#include /* * Go through all existing windows (if the window manager is restarted) and manage them @@ -49,7 +52,7 @@ void manage_existing_windows(xcb_window_t root) { * side-effects which are to be expected when continuing to run i3. * */ -void restore_geometry() { +void restore_geometry(void) { DLOG("Restoring geometry\n"); Con *con; @@ -64,8 +67,41 @@ void restore_geometry() { con->rect.x, con->rect.y); } + /* Strictly speaking, this line doesn’t really belong here, but since we + * are syncing, let’s un-register as a window manager first */ + xcb_change_window_attributes(conn, root, XCB_CW_EVENT_MASK, (uint32_t[]){ XCB_EVENT_MASK_SUBSTRUCTURE_REDIRECT }); + /* Make sure our changes reach the X server, we restart/exit now */ - xcb_flush(conn); + xcb_aux_sync(conn); +} + +/* + * The following function sends a new window event, which consists + * of fields "change" and "container", the latter containing a dump + * of the window's container. + * + */ +static void ipc_send_window_new_event(Con *con) { + setlocale(LC_NUMERIC, "C"); + yajl_gen gen = ygenalloc(); + + y(map_open); + + ystr("change"); + ystr("new"); + + ystr("container"); + dump_node(gen, con, false); + + y(map_close); + + const unsigned char *payload; + ylength length; + y(get_buf, &payload, &length); + + ipc_send_event("window", I3_IPC_EVENT_WINDOW, (const char *)payload); + y(free); + setlocale(LC_NUMERIC, ""); } /* @@ -79,45 +115,38 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki xcb_get_geometry_reply_t *geom; xcb_get_window_attributes_reply_t *attr = NULL; - DLOG("---> looking at window 0x%08x\n", window); - xcb_get_property_cookie_t wm_type_cookie, strut_cookie, state_cookie, utf8_title_cookie, title_cookie, class_cookie, leader_cookie, transient_cookie, - role_cookie, startup_id_cookie; + role_cookie, startup_id_cookie, wm_hints_cookie, + motif_wm_hints_cookie; geomc = xcb_get_geometry(conn, d); -#define FREE_GEOMETRY() do { \ - if ((geom = xcb_get_geometry_reply(conn, geomc, 0)) != NULL) \ - free(geom); \ -} while (0) /* Check if the window is mapped (it could be not mapped when intializing and calling manage_window() for every window) */ if ((attr = xcb_get_window_attributes_reply(conn, cookie, 0)) == NULL) { DLOG("Could not get attributes\n"); - FREE_GEOMETRY(); + xcb_discard_reply(conn, geomc.sequence); return; } if (needs_to_be_mapped && attr->map_state != XCB_MAP_STATE_VIEWABLE) { - DLOG("map_state unviewable\n"); - FREE_GEOMETRY(); + xcb_discard_reply(conn, geomc.sequence); goto out; } /* Don’t manage clients with the override_redirect flag */ - DLOG("override_redirect is %d\n", attr->override_redirect); if (attr->override_redirect) { - FREE_GEOMETRY(); + xcb_discard_reply(conn, geomc.sequence); goto out; } /* Check if the window is already managed */ if (con_by_window_id(window) != NULL) { DLOG("already managed (by con %p)\n", con_by_window_id(window)); - FREE_GEOMETRY(); + xcb_discard_reply(conn, geomc.sequence); goto out; } @@ -130,11 +159,22 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki uint32_t values[1]; /* Set a temporary event mask for the new window, consisting only of - * PropertyChange. We need to be notified of PropertyChanges because the - * client can change its properties *after* we requested them but *before* - * we actually reparented it and have set our final event mask. */ - values[0] = XCB_EVENT_MASK_PROPERTY_CHANGE; - xcb_change_window_attributes(conn, window, XCB_CW_EVENT_MASK, values); + * PropertyChange and StructureNotify. We need to be notified of + * PropertyChanges because the client can change its properties *after* we + * requested them but *before* we actually reparented it and have set our + * final event mask. + * We need StructureNotify because the client may unmap the window before + * we get to re-parent it. + * If this request fails, we assume the client has already unmapped the + * window between the MapRequest and our event mask change. */ + values[0] = XCB_EVENT_MASK_PROPERTY_CHANGE | + XCB_EVENT_MASK_STRUCTURE_NOTIFY; + xcb_void_cookie_t event_mask_cookie = + xcb_change_window_attributes_checked(conn, window, XCB_CW_EVENT_MASK, values); + if (xcb_request_check(conn, event_mask_cookie) != NULL) { + LOG("Could not change event mask, the window probably already disappeared.\n"); + goto out; + } #define GET_PROPERTY(atom, len) xcb_get_property(conn, false, window, atom, XCB_GET_PROPERTY_TYPE_ANY, 0, len) @@ -148,12 +188,15 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki class_cookie = GET_PROPERTY(XCB_ATOM_WM_CLASS, 128); role_cookie = GET_PROPERTY(A_WM_WINDOW_ROLE, 128); startup_id_cookie = GET_PROPERTY(A__NET_STARTUP_ID, 512); + wm_hints_cookie = xcb_icccm_get_wm_hints(conn, window); + motif_wm_hints_cookie = GET_PROPERTY(A__MOTIF_WM_HINTS, 5 * sizeof(uint64_t)); /* TODO: also get wm_normal_hints here. implement after we got rid of xcb-event */ - DLOG("reparenting!\n"); + DLOG("Managing window 0x%08x\n", window); i3Window *cwindow = scalloc(sizeof(i3Window)); cwindow->id = window; + cwindow->depth = get_visual_depth(attr->visual); /* We need to grab the mouse buttons for click to focus */ xcb_grab_button(conn, false, window, XCB_EVENT_MASK_BUTTON_PRESS, @@ -163,9 +206,13 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki xcb_grab_button(conn, false, window, XCB_EVENT_MASK_BUTTON_PRESS, XCB_GRAB_MODE_SYNC, XCB_GRAB_MODE_ASYNC, root, XCB_NONE, - 3 /* right mouse button */, + 2 /* middle mouse button */, XCB_BUTTON_MASK_ANY /* don’t filter for any modifiers */); + xcb_grab_button(conn, false, window, XCB_EVENT_MASK_BUTTON_PRESS, + XCB_GRAB_MODE_SYNC, XCB_GRAB_MODE_ASYNC, root, XCB_NONE, + 3 /* right mouse button */, + XCB_BUTTON_MASK_ANY /* don’t filter for any modifiers */); /* update as much information as possible so far (some replies may be NULL) */ window_update_class(cwindow, xcb_get_property_reply(conn, class_cookie, NULL), true); @@ -175,6 +222,10 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki window_update_transient_for(cwindow, xcb_get_property_reply(conn, transient_cookie, NULL)); window_update_strut_partial(cwindow, xcb_get_property_reply(conn, strut_cookie, NULL)); window_update_role(cwindow, xcb_get_property_reply(conn, role_cookie, NULL), true); + bool urgency_hint; + window_update_hints(cwindow, xcb_get_property_reply(conn, wm_hints_cookie, NULL), &urgency_hint); + border_style_t motif_border_style = BS_NORMAL; + window_update_motif_hints(cwindow, xcb_get_property_reply(conn, motif_wm_hints_cookie, NULL), &motif_border_style); xcb_get_property_reply_t *startup_id_reply; startup_id_reply = xcb_get_property_reply(conn, startup_id_cookie, NULL); @@ -205,7 +256,7 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki cwindow->dock = W_DOCK_BOTTOM; } else { DLOG("Ignoring invalid reserved edges (_NET_WM_STRUT_PARTIAL), using position as fallback:\n"); - if (geom->y < (search_at->rect.height / 2)) { + if (geom->y < (int16_t)(search_at->rect.height / 2)) { DLOG("geom->y = %d < rect.height / 2 = %d, it is a top dock client\n", geom->y, (search_at->rect.height / 2)); cwindow->dock = W_DOCK_TOP; @@ -220,7 +271,7 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki DLOG("Initial geometry: (%d, %d, %d, %d)\n", geom->x, geom->y, geom->width, geom->height); Con *nc = NULL; - Match *match; + Match *match = NULL; Assignment *assignment; /* TODO: two matches for one container */ @@ -232,11 +283,17 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki if ((assignment = assignment_for(cwindow, A_TO_WORKSPACE | A_TO_OUTPUT))) { DLOG("Assignment matches (%p)\n", match); if (assignment->type == A_TO_WORKSPACE) { - nc = con_descend_tiling_focused(workspace_get(assignment->dest.workspace, NULL)); - DLOG("focused on ws %s: %p / %s\n", assignment->dest.workspace, nc, nc->name); + Con *assigned_ws = workspace_get(assignment->dest.workspace, NULL); + nc = con_descend_tiling_focused(assigned_ws); + DLOG("focused on ws %s: %p / %s\n", assigned_ws->name, nc, nc->name); if (nc->type == CT_WORKSPACE) nc = tree_open_con(nc, cwindow); - else nc = tree_open_con(nc->parent, cwindow); + else + nc = tree_open_con(nc->parent, cwindow); + + /* set the urgency hint on the window if the workspace is not visible */ + if (!workspace_is_visible(assigned_ws)) + urgency_hint = true; } /* TODO: handle assignments with type == A_TO_OUTPUT */ } else if (startup_ws) { @@ -262,31 +319,60 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki if (match != NULL && match->insert_where == M_BELOW) { nc = tree_open_con(nc, cwindow); } + + /* If M_BELOW is not used, the container is replaced. This happens with + * "swallows" criteria that are used for stored layouts, in which case + * we need to remove that criterion, because they should only be valid + * once. */ + if (match != NULL && match->insert_where != M_BELOW) { + DLOG("Removing match %p from container %p\n", match, nc); + TAILQ_REMOVE(&(nc->swallow_head), match, matches); + } } DLOG("new container = %p\n", nc); + if (nc->window != NULL && nc->window != cwindow) { + if (!restore_kill_placeholder(nc->window->id)) { + DLOG("Uh?! Container without a placeholder, but with a window, has swallowed this to-be-managed window?!\n"); + } + } nc->window = cwindow; x_reinit(nc); nc->border_width = geom->border_width; char *name; - asprintf(&name, "[i3 con] container around %p", cwindow); + sasprintf(&name, "[i3 con] container around %p", cwindow); x_set_name(nc, name); free(name); + /* handle fullscreen containers */ Con *ws = con_get_workspace(nc); Con *fs = (ws ? con_get_fullscreen_con(ws, CF_OUTPUT) : NULL); if (fs == NULL) fs = con_get_fullscreen_con(croot, CF_GLOBAL); + xcb_get_property_reply_t *state_reply = xcb_get_property_reply(conn, state_cookie, NULL); + if (xcb_reply_contains_atom(state_reply, A__NET_WM_STATE_FULLSCREEN)) { + fs = NULL; + con_toggle_fullscreen(nc, CF_OUTPUT); + } + + FREE(state_reply); + if (fs == NULL) { DLOG("Not in fullscreen mode, focusing\n"); if (!cwindow->dock) { - /* Check that the workspace is visible. If the window was assigned - * to an invisible workspace, we should not steal focus. */ - if (workspace_is_visible(ws)) { - con_focus(nc); + /* Check that the workspace is visible and on the same output as + * the current focused container. If the window was assigned to an + * invisible workspace, we should not steal focus. */ + Con *current_output = con_get_output(focused); + Con *target_output = con_get_output(ws); + + if (workspace_is_visible(ws) && current_output == target_output) { + if (!match || !match->restart_mode) { + con_focus(nc); + } else DLOG("not focusing, matched with restart_mode == true\n"); } else DLOG("workspace not visible, not focusing\n"); } else DLOG("dock, not focusing\n"); } else { @@ -320,13 +406,29 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki (cwindow->leader != XCB_NONE && cwindow->leader != cwindow->id && con_by_window_id(cwindow->leader) != NULL)) { - LOG("This window is transiert for another window, setting floating\n"); + LOG("This window is transient for another window, setting floating\n"); want_floating = true; if (config.popup_during_fullscreen == PDF_LEAVE_FULLSCREEN && fs != NULL) { LOG("There is a fullscreen window, leaving fullscreen mode\n"); con_toggle_fullscreen(fs, CF_OUTPUT); + } else if (config.popup_during_fullscreen == PDF_SMART && + fs != NULL && + fs->window != NULL) { + i3Window *transient_win = cwindow; + while (transient_win != NULL && + transient_win->transient_for != XCB_NONE) { + if (transient_win->transient_for == fs->window->id) { + LOG("This floating window belongs to the fullscreen window (popup_during_fullscreen == smart)\n"); + con_focus(nc); + break; + } + Con *next_transient = con_by_window_id(transient_win->transient_for); + if (next_transient == NULL) + break; + transient_win = next_transient->window; + } } } @@ -346,6 +448,11 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki floating_enable(nc, true); } + if (motif_border_style != BS_NORMAL) { + DLOG("MOTIF_WM_HINTS specifies decorations (border_style = %d)\n", motif_border_style); + con_set_border_style(nc, motif_border_style, config.default_border_width); + } + /* to avoid getting an UnmapNotify event due to reparenting, we temporarily * declare no interest in any state change event of this window */ values[0] = XCB_NONE; @@ -361,12 +468,6 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki xcb_change_window_attributes(conn, window, XCB_CW_EVENT_MASK, values); xcb_flush(conn); - reply = xcb_get_property_reply(conn, state_cookie, NULL); - if (xcb_reply_contains_atom(reply, A__NET_WM_STATE_FULLSCREEN)) - con_toggle_fullscreen(nc, CF_OUTPUT); - - FREE(reply); - /* Put the client inside the save set. Upon termination (whether killed or * normal exit does not matter) of the window manager, these clients will * be correctly reparented to their most closest living ancestor (= @@ -376,8 +477,37 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki /* Check if any assignments match */ run_assignments(cwindow); + /* 'ws' may be invalid because of the assignments, e.g. when the user uses + * "move window to workspace 1", but had it assigned to workspace 2. */ + ws = con_get_workspace(nc); + + /* If this window was put onto an invisible workspace (via assignments), we + * render this workspace. It wouldn’t be rendered in our normal code path + * because only the visible workspaces get rendered. + * + * By rendering the workspace, we assign proper coordinates (read: not + * width=0, height=0) to the window, which is important for windows who + * actually use them to position their GUI elements, e.g. rhythmbox. */ + if (ws && !workspace_is_visible(ws)) { + /* This is a bit hackish: we need to copy the content container’s rect + * to the workspace, because calling render_con() on the content + * container would also take the shortcut and not render the invisible + * workspace at all. However, just calling render_con() on the + * workspace isn’t enough either — it needs the rect. */ + ws->rect = ws->parent->rect; + render_con(ws, true); + } tree_render(); + /* Send an event about window creation */ + ipc_send_window_new_event(nc); + + /* Windows might get managed with the urgency hint already set (Pidgin is + * known to do that), so check for that and handle the hint accordingly. + * This code needs to be in this part of manage_window() because the window + * needs to be on the final workspace first. */ + con_set_urgency(nc, urgency_hint); + geom_out: free(geom); out: