X-Git-Url: https://jasonwoof.com/gitweb/?a=blobdiff_plain;f=spectrwm.c;h=932f11ff7550d15e180d74ad466fa5b3b43491ad;hb=5f8042da0ee96bb4475049d901659e4ca26c625c;hp=648b0c54dc0e47fb0008a1f6d5da128c770340dc;hpb=5d83029a3a9b69ab0f1287aa6c0499b681d891d9;p=spectrwm.git diff --git a/spectrwm.c b/spectrwm.c index 648b0c5..932f11f 100644 --- a/spectrwm.c +++ b/spectrwm.c @@ -128,6 +128,7 @@ static const char *buildstr = SPECTRWM_VERSION; #define XCB_ICCCM_SIZE_HINT_P_MIN_SIZE XCB_SIZE_HINT_P_MIN_SIZE #define XCB_ICCCM_SIZE_HINT_P_MAX_SIZE XCB_SIZE_HINT_P_MAX_SIZE #define XCB_ICCCM_SIZE_HINT_P_RESIZE_INC XCB_SIZE_HINT_P_RESIZE_INC +#define XCB_ICCCM_WM_HINT_INPUT XCB_WM_HINT_INPUT #define XCB_ICCCM_WM_HINT_X_URGENCY XCB_WM_HINT_X_URGENCY #define XCB_ICCCM_WM_STATE_ICONIC XCB_WM_STATE_ICONIC #define XCB_ICCCM_WM_STATE_WITHDRAWN XCB_WM_STATE_WITHDRAWN @@ -490,6 +491,7 @@ struct workspace { struct layout *cur_layout; /* current layout handlers */ struct ws_win *focus; /* may be NULL */ struct ws_win *focus_prev; /* may be NULL */ + struct ws_win *focus_pending; /* may be NULL */ struct swm_region *r; /* may be NULL */ struct swm_region *old_r; /* may be NULL */ struct ws_win_list winlist; /* list of windows in ws */ @@ -849,7 +851,6 @@ get_swm_iconic(struct ws_win *win) int32_t v = 0; xcb_get_property_reply_t *pr = NULL; - pr = xcb_get_property_reply(conn, xcb_get_property(conn, 0, win->id, a_swm_iconic, XCB_ATOM_INTEGER, 0, 1), NULL); @@ -2847,6 +2848,8 @@ switchws(struct swm_region *r, union arg *args) if (new_ws == old_ws) return; + unfocus_win(old_ws->focus); + other_r = new_ws->r; if (other_r == NULL) { /* the other workspace is hidden, hide this one */ @@ -2862,8 +2865,6 @@ switchws(struct swm_region *r, union arg *args) this_r->ws = new_ws; new_ws->r = this_r; - unfocus_win(old_ws->focus); - stack(); /* unmap old windows */ @@ -2871,16 +2872,16 @@ switchws(struct swm_region *r, union arg *args) TAILQ_FOREACH(win, &old_ws->winlist, entry) unmap_window(win); - new_ws->focus = get_region_focus(new_ws->r); + new_ws->focus_pending = get_region_focus(new_ws->r); - /* unmap old windows */ - if (unmap_old) - TAILQ_FOREACH(win, &old_ws->winlist, entry) - unmap_window(win); - - /* make sure bar gets updated if ws is empty */ - if (!new_ws->focus) + if (new_ws->focus_pending) { + /* if workspaces were swapped, then don't wait to set focus */ + if (old_ws->r) + focus_win(new_ws->focus); + } else { + /* make sure bar gets updated if ws is empty */ bar_update(); + } focus_flush(); @@ -3027,7 +3028,6 @@ swapwin(struct swm_region *r, union arg *args) struct ws_win *cur_focus; struct ws_win_list *wl; - DNPRINTF(SWM_D_WS, "swapwin: id: %d, screen[%d]:%dx%d+%d+%d, ws: %d\n", args->id, r->s->idx, WIDTH(r), HEIGHT(r), X(r), Y(r), r->ws->idx); @@ -3521,7 +3521,7 @@ stack_master(struct workspace *ws, struct swm_geometry *g, int rot, int flip) int w_inc = 1, h_inc, w_base = 1, h_base; int hrh, extra = 0, h_slice, last_h = 0; int split, colno, winno, mwin, msize, mscale; - int remain, missing, v_slice, reconfigure; + int remain, missing, v_slice, reconfigure = 0; int bordered = 1; DNPRINTF(SWM_D_STACK, "stack_master: workspace: %d, rot: %s, " @@ -4432,7 +4432,6 @@ wkill(struct swm_region *r, union arg *args) xcb_flush(conn); } - int floating_toggle_win(struct ws_win *win) { @@ -4896,7 +4895,6 @@ move_step(struct swm_region *r, union arg *args) move(win, args); } - /* user/key callable function IDs */ enum keyfuncid { KF_BAR_TOGGLE, @@ -6505,7 +6503,6 @@ struct config_option configopt[] = { { "layout", setlayout, 0 }, }; - int conf_load(char *filename, int keymapping) { @@ -6513,7 +6510,7 @@ conf_load(char *filename, int keymapping) char *line, *cp, *optsub, *optval; size_t linelen, lineno = 0; int wordlen, i, optidx; - struct config_option *opt; + struct config_option *opt = NULL; DNPRINTF(SWM_D_CONF, "conf_load: begin\n"); @@ -6562,7 +6559,7 @@ conf_load(char *filename, int keymapping) filename, lineno, wordlen, cp); goto out; } - if (keymapping && strcmp(opt->optname, "bind")) { + if (keymapping && opt && strcmp(opt->optname, "bind")) { warnx("%s: line %zd: invalid option %.*s", filename, lineno, wordlen, cp); goto out; @@ -6914,7 +6911,6 @@ manage_window(xcb_window_t id, uint16_t mapped) update_window(win); } - /* Select which X events to monitor and set border pixel color. */ wa[0] = win->s->c[SWM_S_COLOR_UNFOCUS].pixel; wa[1] = XCB_EVENT_MASK_ENTER_WINDOW | XCB_EVENT_MASK_PROPERTY_CHANGE | @@ -7001,8 +6997,6 @@ unmanage_window(struct ws_win *win) parent->focus_child = NULL; } - focus_win(get_focus_prev(win)); - TAILQ_REMOVE(&win->ws->winlist, win, entry); TAILQ_INSERT_TAIL(&win->ws->unmanagedlist, win, entry); } @@ -7306,14 +7300,20 @@ destroynotify(xcb_destroy_notify_event_t *e) return; } - /* make sure we focus on something */ - win->floating = 0; + /* If we were focused, make sure we focus on something else. */ + if (win == win->ws->focus) + win->ws->focus_pending = get_focus_prev(win); unmanage_window(win); - free_window(win); - stack(); + if (win->ws->focus_pending) { + focus_win(win->ws->focus_pending); + win->ws->focus_pending = NULL; + } + + free_window(win); + focus_flush(); } @@ -7444,9 +7444,10 @@ mapnotify(xcb_map_notify_event_t *e) win->mapped = 1; set_win_state(win, XCB_ICCCM_WM_STATE_NORMAL); - /* Focus on window if it is selected. */ - if (win->ws->focus == win) + if (win->ws->focus_pending == win) { focus_win(win); + win->ws->focus_pending = NULL; + } xcb_flush(conn); } @@ -7491,7 +7492,7 @@ maprequest(xcb_map_request_event_t *e) stack(); /* The new window should get focus. */ - win->ws->focus = get_focus_magic(win); + win->ws->focus_pending = get_focus_magic(win); /* Ignore EnterNotify to handle the mapnotify without interference. */ if (focus_mode == SWM_FOCUS_DEFAULT) @@ -7556,30 +7557,30 @@ propertynotify(xcb_property_notify_event_t *e) if (e->atom == a_swm_iconic) { if (e->state == XCB_PROPERTY_DELETE) { /* The window is no longer iconic, restack ws. */ + win->ws->focus_pending = get_focus_magic(win); stack(); - /* The window should get focus. */ - win->ws->focus = get_focus_magic(win); - /* Flush EnterNotify for mapnotify, if needed. */ focus_flush(); return; } else if (e->state == XCB_PROPERTY_NEW_VALUE) { - win->ws->focus = NULL; - + win->ws->focus_pending = get_focus_prev(win); unfocus_win(win); unmap_window(win); if (win->ws->r) { - focus_win(get_focus_prev(win)); stack(); + focus_win(win->ws->focus_pending); + win->ws->focus_pending = NULL; focus_flush(); } } } else if (e->atom == a_state && e->state == XCB_PROPERTY_NEW_VALUE) { - /* Focus on window if it is selected. */ - if (win->ws->focus == win) + /* State just changed, make sure it gets focused if mapped. */ + if (win->mapped && win->ws->focus_pending == win) { + win->ws->focus_pending = NULL; focus_win(win); + } } switch (e->atom) { @@ -7607,8 +7608,18 @@ unmapnotify(xcb_unmap_notify_event_t *e) return; if (getstate(e->window) == XCB_ICCCM_WM_STATE_NORMAL) { + /* If we were focused, make sure we focus on something else. */ + if (win == win->ws->focus) + win->ws->focus_pending = get_focus_prev(win); + unmanage_window(win); stack(); + + if (win->ws->focus_pending) { + focus_win(win->ws->focus_pending); + win->ws->focus_pending = NULL; + } + focus_flush(); } } @@ -8085,6 +8096,8 @@ setup_screens(void) ws->idx = j; ws->name = NULL; ws->focus = NULL; + ws->focus_prev = NULL; + ws->focus_pending = NULL; ws->r = NULL; ws->old_r = NULL; TAILQ_INIT(&ws->winlist);