X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;ds=sidebyside;f=openbox%2Ffocus_cycle.c;h=bd8a08465e95ef47aff9b01df6551441e14f7d37;hb=4c56b2de8d649764755036c1ff8c0618c22fb468;hp=7549fbe51e2dce5905a976ff1dadcf360d9f9dae;hpb=461ae2a83349b7574df5e65e8238704f414891d1;p=chaz%2Fopenbox diff --git a/openbox/focus_cycle.c b/openbox/focus_cycle.c index 7549fbe5..bd8a0846 100644 --- a/openbox/focus_cycle.c +++ b/openbox/focus_cycle.c @@ -44,10 +44,6 @@ static ObClient *focus_find_directional (ObClient *c, ObDirection dir, gboolean dock_windows, gboolean desktop_windows); -static ObClient *focus_find_directional (ObClient *c, - ObDirection dir, - gboolean dock_windows, - gboolean desktop_windows); void focus_cycle_startup(gboolean reconfig) { @@ -89,7 +85,7 @@ static gboolean focus_target_has_siblings(ObClient *ft, for (it = ft->group->members; it; it = g_slist_next(it)) { ObClient *c = it->data; /* check that it's not a helper window to avoid infinite recursion */ - if (c != ft && !client_helper(c) && + if (c != ft && c->type == OB_CLIENT_TYPE_NORMAL && focus_cycle_target_valid(c, iconic_windows, all_desktops, FALSE, FALSE)) { @@ -131,7 +127,7 @@ gboolean focus_cycle_target_valid(ObClient *ft, ok = ok && ((client_normal(ft) && !client_helper(ft)) || - /* helper windows are valid targets it... */ + /* helper windows are valid targets if... */ (client_helper(ft) && /* ...a window in its group already has focus ... */ ((focus_client && ft->group == focus_client->group) || @@ -169,7 +165,6 @@ void focus_cycle(gboolean forward, gboolean all_desktops, gboolean linear, gboolean interactive, gboolean dialog, gboolean done, gboolean cancel) { - static ObClient *first = NULL; static ObClient *t = NULL; static GList *order = NULL; GList *it, *start, *list; @@ -185,8 +180,6 @@ void focus_cycle(gboolean forward, gboolean all_desktops, if (!focus_order) goto done_cycle; - if (!first) first = focus_client; - if (linear) list = client_list; else list = focus_order; } else { @@ -201,10 +194,10 @@ void focus_cycle(gboolean forward, gboolean all_desktops, focus_cycle_all_desktops = all_desktops; focus_cycle_dock_windows = dock_windows; focus_cycle_desktop_windows = desktop_windows; - focus_cycle_target = focus_client; - } + start = it = g_list_find(list, focus_client); + } else + start = it = g_list_find(list, focus_cycle_target); - start = it = g_list_find(list, focus_cycle_target); if (!start) /* switched desktops or something? */ start = it = forward ? g_list_last(list) : g_list_first(list); if (!start) goto done_cycle; @@ -250,7 +243,6 @@ done_cycle: client_activate(focus_cycle_target, FALSE, TRUE); t = NULL; - first = NULL; focus_cycle_target = NULL; g_list_free(order); order = NULL; @@ -275,7 +267,7 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, ObClient *best_client, *cur; GList *it; - if(!client_list) + if (!client_list) return NULL; /* first, find the centre coords of the currently focused window */ @@ -283,13 +275,13 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, my_cy = c->frame->area.y + c->frame->area.height / 2; best_score = -1; - best_client = NULL; + best_client = c; - for(it = g_list_first(client_list); it; it = g_list_next(it)) { + for (it = g_list_first(client_list); it; it = g_list_next(it)) { cur = it->data; /* the currently selected window isn't interesting */ - if(cur == c) + if (cur == c) continue; if (!focus_cycle_target_valid(it->data, FALSE, FALSE, dock_windows, desktop_windows)) @@ -302,8 +294,9 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, his_cy = (cur->frame->area.y - my_cy) + cur->frame->area.height / 2; - if(dir == OB_DIRECTION_NORTHEAST || dir == OB_DIRECTION_SOUTHEAST || - dir == OB_DIRECTION_SOUTHWEST || dir == OB_DIRECTION_NORTHWEST) { + if (dir == OB_DIRECTION_NORTHEAST || dir == OB_DIRECTION_SOUTHEAST || + dir == OB_DIRECTION_SOUTHWEST || dir == OB_DIRECTION_NORTHWEST) + { gint tx; /* Rotate the diagonals 45 degrees counterclockwise. * To do this, multiply the matrix /+h +h\ with the @@ -315,7 +308,7 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, his_cx = tx; } - switch(dir) { + switch (dir) { case OB_DIRECTION_NORTH: case OB_DIRECTION_SOUTH: case OB_DIRECTION_NORTHEAST: @@ -337,7 +330,7 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, } /* the target must be in the requested direction */ - if(distance <= 0) + if (distance <= 0) continue; /* Calculate score for this window. The smaller the better. */ @@ -346,12 +339,13 @@ static ObClient *focus_find_directional(ObClient *c, ObDirection dir, /* windows more than 45 degrees off the direction are * heavily penalized and will only be chosen if nothing * else within a million pixels */ - if(offset > distance) + if (offset > distance) score += 1000000; - if(best_score == -1 || score < best_score) - best_client = cur, - best_score = score; + if (best_score == -1 || score < best_score) { + best_client = cur; + best_score = score; + } } return best_client; @@ -364,13 +358,10 @@ void focus_directional_cycle(ObDirection dir, gboolean dock_windows, static ObClient *first = NULL; ObClient *ft = NULL; - if (!interactive) - return; - if (cancel) { focus_cycle_target = NULL; goto done_cycle; - } else if (done) + } else if (done && interactive) goto done_cycle; if (!focus_order) @@ -381,7 +372,6 @@ void focus_directional_cycle(ObDirection dir, gboolean dock_windows, focus_cycle_all_desktops = FALSE; focus_cycle_dock_windows = dock_windows; focus_cycle_desktop_windows = desktop_windows; - focus_cycle_target = focus_client; } if (!first) first = focus_client; @@ -389,6 +379,8 @@ void focus_directional_cycle(ObDirection dir, gboolean dock_windows, if (focus_cycle_target) ft = focus_find_directional(focus_cycle_target, dir, dock_windows, desktop_windows); + else if (first) + ft = focus_find_directional(first, dir, dock_windows, desktop_windows); else { GList *it; @@ -401,21 +393,20 @@ void focus_directional_cycle(ObDirection dir, gboolean dock_windows, ft = it->data; } - if (ft) { - if (ft != focus_cycle_target) {/* prevents flicker */ - focus_cycle_target = ft; - focus_cycle_draw_indicator(ft); - } + if (ft && ft != focus_cycle_target) {/* prevents flicker */ + focus_cycle_target = ft; + if (!interactive) + goto done_cycle; + focus_cycle_draw_indicator(ft); } - if (focus_cycle_target && dialog) { + if (focus_cycle_target && dialog) /* same arguments as focus_target_valid */ focus_cycle_popup_single_show(focus_cycle_target, focus_cycle_iconic_windows, focus_cycle_all_desktops, focus_cycle_dock_windows, focus_cycle_desktop_windows); - return; - } + return; done_cycle: if (done && focus_cycle_target)