X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fplace.c;h=7d5c869439863e7cd594b02aa94e72c074af8668;hb=HEAD;hp=a098921d8792bd1b0a0e4a737a7df8109cac14c8;hpb=bdf5d409e5c10cc4b37658491ccbba75978c5141;p=chaz%2Fopenbox diff --git a/openbox/place.c b/openbox/place.c index a098921d..7d5c8694 100644 --- a/openbox/place.c +++ b/openbox/place.c @@ -1,7 +1,8 @@ /* -*- indent-tabs-mode: nil; tab-width: 4; c-basic-offset: 4; -*- place.c for the Openbox window manager - Copyright (c) 2003 Ben Jansens + Copyright (c) 2006 Mikael Magnusson + Copyright (c) 2003-2007 Dana Jansens This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -22,381 +23,489 @@ #include "frame.h" #include "focus.h" #include "config.h" +#include "dock.h" +#include "debug.h" +#include "place_overlap.h" -static Rect* pick_head(ObClient *c) +static Rect *choose_pointer_monitor(ObClient *c) { - /* try direct parent first */ - if (c->transient_for && c->transient_for != OB_TRAN_GROUP) { - return screen_area_monitor(c->desktop, - client_monitor(c->transient_for)); - } + return screen_area(c->desktop, screen_monitor_pointer(), NULL); +} - /* more than one guy in his group (more than just him) */ - if (client_has_group_siblings(c)) { - GSList *it; +/* use the following priority lists for choose_monitor() + + When a window is being placed in the FOREGROUND, use a monitor chosen in + the following order: + 1. per-app settings + 2. same monitor as parent + 3. primary monitor if placement=PRIMARY + active monitor if placement=ACTIVE + pointer monitor if placement=MOUSE + 4. primary monitor + 5. other monitors where the window has group members on the same desktop + 6. other monitors where the window has group members on other desktops + 7. other monitors + + When a window is being placed in the BACKGROUND, use a monitor chosen in the + following order: + 1. per-app settings + 2. same monitor as parent + 3. other monitors where the window has group members on the same desktop + 3a. primary monitor in this set + 3b. other monitors in this set + 4. other monitors where the window has group members on other desktops + 4a. primary monitor in this set + 4b. other monitors in this set + 5. other monitors + 5a. primary monitor in this set + 5b. other monitors in this set +*/ - /* try on the client's desktop */ - for (it = c->group->members; it; it = g_slist_next(it)) { - ObClient *itc = it->data; - if (itc != c && - (itc->desktop == c->desktop || - itc->desktop == DESKTOP_ALL || c->desktop == DESKTOP_ALL)) - return screen_area_monitor(c->desktop, - client_monitor(it->data)); - } +/*! One for each possible head, used to sort them in order of precedence. */ +typedef struct { + guint monitor; + guint flags; +} ObPlaceHead; + +/*! Flags for ObPlaceHead */ +enum { + HEAD_PARENT = 1 << 0, /* parent's monitor */ + HEAD_PLACED = 1 << 1, /* chosen monitor by placement */ + HEAD_PRIMARY = 1 << 2, /* primary monitor */ + HEAD_GROUP_DESK = 1 << 3, /* has a group member on the same desktop */ + HEAD_GROUP = 1 << 4, /* has a group member on another desktop */ + HEAD_PERAPP = 1 << 5, /* chosen by per-app settings */ +}; + +gint cmp_foreground(const void *a, const void *b) +{ + const ObPlaceHead *h1 = a; + const ObPlaceHead *h2 = b; + gint i = 0; - /* try on all desktops */ - for (it = c->group->members; it; it = g_slist_next(it)) { - ObClient *itc = it->data; - if (itc != c) - return screen_area_monitor(c->desktop, - client_monitor(it->data)); - } - } + if (h1->monitor == h2->monitor) return 0; - return NULL; -} + if (h1->flags & HEAD_PERAPP) --i; + if (h2->flags & HEAD_PERAPP) ++i; + if (i) return i; -static gboolean place_random(ObClient *client, gint *x, gint *y) -{ - gint l, r, t, b; - Rect *area; + if (h1->flags & HEAD_PARENT) --i; + if (h2->flags & HEAD_PARENT) ++i; + if (i) return i; - area = pick_head(client); - if (!area) - area = screen_area_monitor(client->desktop, - g_random_int_range(0, screen_num_monitors)); + if (h1->flags & HEAD_PLACED) --i; + if (h2->flags & HEAD_PLACED) ++i; + if (i) return i; - l = area->x; - t = area->y; - r = area->x + area->width - client->frame->area.width; - b = area->y + area->height - client->frame->area.height; + if (h1->flags & HEAD_PRIMARY) --i; + if (h2->flags & HEAD_PRIMARY) ++i; + if (i) return i; - if (r > l) *x = g_random_int_range(l, r + 1); - else *x = 0; - if (b > t) *y = g_random_int_range(t, b + 1); - else *y = 0; + if (h1->flags & HEAD_GROUP_DESK) --i; + if (h2->flags & HEAD_GROUP_DESK) ++i; + if (i) return i; - return TRUE; -} + if (h1->flags & HEAD_GROUP) --i; + if (h2->flags & HEAD_GROUP) ++i; + if (i) return i; -static GSList* area_add(GSList *list, Rect *a) -{ - Rect *r = g_new(Rect, 1); - *r = *a; - return g_slist_prepend(list, r); + return h1->monitor - h2->monitor; } -static GSList* area_remove(GSList *list, Rect *a) +gint cmp_background(const void *a, const void *b) { - GSList *sit; - GSList *result = NULL; - - for (sit = list; sit; sit = g_slist_next(sit)) { - Rect *r = sit->data; + const ObPlaceHead *h1 = a; + const ObPlaceHead *h2 = b; + gint i = 0; + + if (h1->monitor == h2->monitor) return 0; + + if (h1->flags & HEAD_PERAPP) --i; + if (h2->flags & HEAD_PERAPP) ++i; + if (i) return i; + + if (h1->flags & HEAD_PARENT) --i; + if (h2->flags & HEAD_PARENT) ++i; + if (i) return i; + + if (h1->flags & HEAD_GROUP_DESK || h2->flags & HEAD_GROUP_DESK) { + if (h1->flags & HEAD_GROUP_DESK) --i; + if (h2->flags & HEAD_GROUP_DESK) ++i; + if (i) return i; + if (h1->flags & HEAD_PRIMARY) --i; + if (h2->flags & HEAD_PRIMARY) ++i; + if (i) return i; + } - if (!RECT_INTERSECTS_RECT(*r, *a)) { - result = g_slist_prepend(result, r); - r = NULL; /* dont free it */ - } else { - Rect isect, extra; + if (h1->flags & HEAD_GROUP || h2->flags & HEAD_GROUP) { + if (h1->flags & HEAD_GROUP) --i; + if (h2->flags & HEAD_GROUP) ++i; + if (i) return i; + if (h1->flags & HEAD_PRIMARY) --i; + if (h2->flags & HEAD_PRIMARY) ++i; + if (i) return i; + } - /* Use an intersection of a and r to determine the space - around r that we can use. + if (h1->flags & HEAD_PRIMARY) --i; + if (h2->flags & HEAD_PRIMARY) ++i; + if (i) return i; - NOTE: the spaces calculated can overlap. - */ + return h1->monitor - h2->monitor; +} - RECT_SET_INTERSECTION(isect, *r, *a); +/*! Pick a monitor to place a window on. */ +static Rect* choose_monitor(ObClient *c, gboolean client_to_be_foregrounded, + ObAppSettings *settings) +{ + Rect *area; + ObPlaceHead *choice; + guint i; + ObClient *p; + GSList *it; - if (RECT_LEFT(isect) > RECT_LEFT(*r)) { - RECT_SET(extra, r->x, r->y, - RECT_LEFT(isect) - r->x, r->height); - result = area_add(result, &extra); - } + choice = g_new(ObPlaceHead, screen_num_monitors); + for (i = 0; i < screen_num_monitors; ++i) { + choice[i].monitor = i; + choice[i].flags = 0; + } - if (RECT_TOP(isect) > RECT_TOP(*r)) { - RECT_SET(extra, r->x, r->y, - r->width, RECT_TOP(isect) - r->y + 1); - result = area_add(result, &extra); + /* find monitors with group members */ + if (c->group) { + for (it = c->group->members; it; it = g_slist_next(it)) { + ObClient *itc = it->data; + if (itc != c) { + guint m = client_monitor(itc); + + if (m < screen_num_monitors) { + if (screen_compare_desktops(itc->desktop, c->desktop)) + choice[m].flags |= HEAD_GROUP_DESK; + else + choice[m].flags |= HEAD_GROUP; + } } + } + } - if (RECT_RIGHT(isect) < RECT_RIGHT(*r)) { - RECT_SET(extra, RECT_RIGHT(isect) + 1, r->y, - RECT_RIGHT(*r) - RECT_RIGHT(isect), r->height); - result = area_add(result, &extra); - } + i = screen_monitor_primary(FALSE); + if (i < screen_num_monitors) { + choice[i].flags |= HEAD_PRIMARY; + if (config_place_monitor == OB_PLACE_MONITOR_PRIMARY) + choice[i].flags |= HEAD_PLACED; + if (settings && + settings->monitor_type == OB_PLACE_MONITOR_PRIMARY) + choice[i].flags |= HEAD_PERAPP; + } - if (RECT_BOTTOM(isect) < RECT_BOTTOM(*r)) { - RECT_SET(extra, r->x, RECT_BOTTOM(isect) + 1, - r->width, RECT_BOTTOM(*r) - RECT_BOTTOM(isect)); - result = area_add(result, &extra); - } - } + i = screen_monitor_active(); + if (i < screen_num_monitors) { + if (config_place_monitor == OB_PLACE_MONITOR_ACTIVE) + choice[i].flags |= HEAD_PLACED; + if (settings && + settings->monitor_type == OB_PLACE_MONITOR_ACTIVE) + choice[i].flags |= HEAD_PERAPP; + } - g_free(r); + i = screen_monitor_pointer(); + if (i < screen_num_monitors) { + if (config_place_monitor == OB_PLACE_MONITOR_MOUSE) + choice[i].flags |= HEAD_PLACED; + if (settings && + settings->monitor_type == OB_PLACE_MONITOR_MOUSE) + choice[i].flags |= HEAD_PERAPP; } - g_slist_free(list); - return result; -} -static gint area_cmp(gconstpointer p1, gconstpointer p2, gpointer data) -{ - ObClient *c = data; - Rect *carea = &c->frame->area; - const Rect *a1 = p1, *a2 = p2; - gboolean diffhead = FALSE; - guint i; - Rect *a; + if (settings) { + i = settings->monitor - 1; + if (i < screen_num_monitors) + choice[i].flags |= HEAD_PERAPP; + } - for (i = 0; i < screen_num_monitors; ++i) { - a = screen_physical_area_monitor(i); - if (RECT_CONTAINS(*a, a1->x, a1->y) && - !RECT_CONTAINS(*a, a2->x, a2->y)) - { - diffhead = TRUE; - break; - } + /* direct parent takes highest precedence */ + if ((p = client_direct_parent(c))) { + i = client_monitor(p); + if (i < screen_num_monitors) + choice[i].flags |= HEAD_PARENT; } - /* has to be more than me in the group */ - if (diffhead && client_has_group_siblings(c)) { - guint *num, most; - GSList *it; + qsort(choice, screen_num_monitors, sizeof(ObPlaceHead), + client_to_be_foregrounded ? cmp_foreground : cmp_background); - /* find how many clients in the group are on each monitor, use the - monitor with the most in it */ - num = g_new0(guint, screen_num_monitors); - for (it = c->group->members; it; it = g_slist_next(it)) - if (it->data != c) - ++num[client_monitor(it->data)]; - most = 0; - for (i = 1; i < screen_num_monitors; ++i) - if (num[i] > num[most]) - most = i; - - a = screen_physical_area_monitor(most); - if (RECT_CONTAINS(*a, a1->x, a1->y)) - return -1; - if (RECT_CONTAINS(*a, a2->x, a2->y)) - return 1; + /* save the areas of the monitors in order of their being chosen */ + for (i = 0; i < screen_num_monitors; ++i) + { + ob_debug("placement choice %d is monitor %d", i, choice[i].monitor); + if (choice[i].flags & HEAD_PARENT) + ob_debug(" - parent on monitor"); + if (choice[i].flags & HEAD_PLACED) + ob_debug(" - placement choice"); + if (choice[i].flags & HEAD_PRIMARY) + ob_debug(" - primary monitor"); + if (choice[i].flags & HEAD_GROUP_DESK) + ob_debug(" - group on same desktop"); + if (choice[i].flags & HEAD_GROUP) + ob_debug(" - group on other desktop"); } - return MIN((a1->width - carea->width), (a1->height - carea->height)) - - MIN((a2->width - carea->width), (a2->height - carea->height)); + area = screen_area(c->desktop, choice[0].monitor, NULL); + + g_free(choice); + + /* return the area for the chosen monitor */ + return area; } -typedef enum +static gboolean place_under_mouse(ObClient *client, gint *x, gint *y, + Size frame_size) { - SMART_FULL, - SMART_GROUP, - SMART_FOCUSED -} ObSmartType; - -#define SMART_IGNORE(placer, c) \ - (placer == c || !c->frame->visible || c->shaded || !client_normal(c) || \ - (c->desktop != DESKTOP_ALL && \ - c->desktop != (placer->desktop == DESKTOP_ALL ? \ - screen_desktop : placer->desktop))) - -static gboolean place_smart(ObClient *client, gint *x, gint *y, - ObSmartType type) -{ - guint i; - gboolean ret = FALSE; - GSList *spaces = NULL, *sit; - GList *it; + if (config_place_policy != OB_PLACE_POLICY_MOUSE) + return FALSE; - for (i = 0; i < screen_num_monitors; ++i) - spaces = area_add(spaces, screen_area_monitor(client->desktop, i)); + gint l, r, t, b; + gint px, py; + Rect *area; - /* stay out from under windows in higher layers */ - for (it = stacking_list; it; it = g_list_next(it)) { - ObClient *c; + ob_debug("placing under mouse"); - if (WINDOW_IS_CLIENT(it->data)) { - c = it->data; - if (c->fullscreen) - continue; - } else - continue; - - if (c->layer > client->layer) { - if (!SMART_IGNORE(client, c)) - spaces = area_remove(spaces, &c->frame->area); - } else - break; - } + if (!screen_pointer_pos(&px, &py)) + return FALSE; + area = choose_pointer_monitor(client); - if (client->type == OB_CLIENT_TYPE_NORMAL) { - if (type == SMART_FULL || type == SMART_FOCUSED) { - gboolean found_foc = FALSE, stop = FALSE; - ObClient *foc; - GList *list; + l = area->x; + t = area->y; + r = area->x + area->width - frame_size.width; + b = area->y + area->height - frame_size.height; - list = focus_order[client->desktop == DESKTOP_ALL ? - screen_desktop : client->desktop]; - foc = list ? list->data : NULL; + *x = px - frame_size.width / 2; + *x = MIN(MAX(*x, l), r); + *y = py - frame_size.height / 2; + *y = MIN(MAX(*y, t), b); - for (; it && !stop; it = g_list_next(it)) { - ObClient *c; + g_slice_free(Rect, area); - if (WINDOW_IS_CLIENT(it->data)) { - c = it->data; - if (c->fullscreen) - continue; - } else - continue; + return TRUE; +} - if (!SMART_IGNORE(client, c)) { - if (type == SMART_FOCUSED) - if (found_foc) - stop = TRUE; - if (!stop) - spaces = area_remove(spaces, &c->frame->area); - } +static gboolean place_per_app_setting_position(ObClient *client, Rect *screen, + gint *x, gint *y, + ObAppSettings *settings, + Size frame_size) +{ + if (!settings || !settings->pos_given) + return FALSE; + + ob_debug("placing by per-app settings"); + + if (settings->position.x.center) + *x = screen->x + screen->width / 2 - client->area.width / 2; + else if (settings->position.x.opposite) + *x = screen->x + screen->width - frame_size.width - + settings->position.x.pos; + else + *x = screen->x + settings->position.x.pos; + if (settings->position.x.denom) + *x = (*x * screen->width) / settings->position.x.denom; + + if (settings->position.y.center) + *y = screen->y + screen->height / 2 - client->area.height / 2; + else if (settings->position.y.opposite) + *y = screen->y + screen->height - frame_size.height - + settings->position.y.pos; + else + *y = screen->y + settings->position.y.pos; + if (settings->position.y.denom) + *y = (*y * screen->height) / settings->position.y.denom; - if (c == foc) - found_foc = TRUE; - } - } else if (type == SMART_GROUP) { - /* has to be more than me in the group */ - if (!client_has_group_siblings(client)) - return FALSE; - - for (sit = client->group->members; sit; sit = g_slist_next(sit)) { - ObClient *c = sit->data; - if (!SMART_IGNORE(client, c)) - spaces = area_remove(spaces, &c->frame->area); - } - } else - g_assert_not_reached(); - } + return TRUE; +} - spaces = g_slist_sort_with_data(spaces, area_cmp, client); +static void place_per_app_setting_size(ObClient *client, Rect *screen, + gint *w, gint *h, + ObAppSettings *settings) +{ + if (!settings) + return; + + g_assert(settings->width_num >= 0); + g_assert(settings->width_denom >= 0); + g_assert(settings->height_num >= 0); + g_assert(settings->height_denom >= 0); + + if (settings->width_num) { + ob_debug("setting width by per-app settings"); + if (!settings->width_denom) + *w = settings->width_num; + else { + *w = screen->width * settings->width_num / settings->width_denom; + *w = MIN(*w, screen->width); + } + } - for (sit = spaces; sit; sit = g_slist_next(sit)) { - Rect *r = sit->data; + if (settings->height_num) { + ob_debug("setting height by per-app settings"); + if (!settings->height_denom) + *h = settings->height_num; + else { + *h = screen->height * settings->height_num / settings->height_denom; + *h = MIN(*h, screen->height); + } + } +} - if (!ret) { - if (r->width >= client->frame->area.width && - r->height >= client->frame->area.height) { - ret = TRUE; - if (client->type == OB_CLIENT_TYPE_DIALOG || - type != SMART_FULL) - { - *x = r->x + (r->width - client->frame->area.width) / 2; - *y = r->y + (r->height - client->frame->area.height) / 2; +static gboolean place_transient_splash(ObClient *client, Rect *area, + gint *x, gint *y, Size frame_size) +{ + if (client->type == OB_CLIENT_TYPE_DIALOG) { + GSList *it; + gboolean first = TRUE; + gint l, r, t, b; + + ob_debug("placing dialog"); + + for (it = client->parents; it; it = g_slist_next(it)) { + ObClient *m = it->data; + if (!m->iconic) { + if (first) { + l = RECT_LEFT(m->frame->area); + t = RECT_TOP(m->frame->area); + r = RECT_RIGHT(m->frame->area); + b = RECT_BOTTOM(m->frame->area); + first = FALSE; } else { - *x = r->x; - *y = r->y; + l = MIN(l, RECT_LEFT(m->frame->area)); + t = MIN(t, RECT_TOP(m->frame->area)); + r = MAX(r, RECT_RIGHT(m->frame->area)); + b = MAX(b, RECT_BOTTOM(m->frame->area)); } } + if (!first) { + *x = ((r + 1 - l) - frame_size.width) / 2 + l; + *y = ((b + 1 - t) - frame_size.height) / 2 + t; + return TRUE; + } } + } + + if (client->type == OB_CLIENT_TYPE_DIALOG || + client->type == OB_CLIENT_TYPE_SPLASH) + { + ob_debug("placing dialog or splash"); - g_free(r); + *x = (area->width - frame_size.width) / 2 + area->x; + *y = (area->height - frame_size.height) / 2 + area->y; + return TRUE; } - g_slist_free(spaces); - return ret; + return FALSE; } -static gboolean place_under_mouse(ObClient *client, gint *x, gint *y) +static gboolean place_least_overlap(ObClient *c, Rect *head, int *x, int *y, + Size frame_size) { - guint i; - gint l, r, t, b; - gint px, py; - Rect *area; - - screen_pointer_pos(&px, &py); + /* Assemble the list of windows that could overlap with @c in the user's + current view. */ + GSList* potential_overlap_clients = NULL; + gint n_client_rects = config_dock_hide ? 0 : 1; + + /* if we're "showing desktop", ignore all existing windows */ + if (!screen_showing_desktop) { + GList* it; + for (it = client_list; it != NULL; it = g_list_next(it)) { + ObClient* maybe_client = (ObClient*)it->data; + if (maybe_client == c) + continue; + if (maybe_client->iconic) + continue; + if (!client_occupies_space(maybe_client)) + continue; + if (c->desktop != DESKTOP_ALL) { + if (maybe_client->desktop != c->desktop && + maybe_client->desktop != DESKTOP_ALL) + continue; + } else { + if (maybe_client->desktop != screen_desktop && + maybe_client->desktop != DESKTOP_ALL) + continue; + } - for (i = 0; i < screen_num_monitors; ++i) { - area = screen_area_monitor(client->desktop, i); - if (RECT_CONTAINS(*area, px, py)) - break; + potential_overlap_clients = g_slist_prepend( + potential_overlap_clients, maybe_client); + n_client_rects += 1; + } } - if (i == screen_num_monitors) - area = screen_area_monitor(client->desktop, 0); - - l = area->x; - t = area->y; - r = area->x + area->width - client->frame->area.width; - b = area->y + area->height - client->frame->area.height; + Rect client_rects[n_client_rects]; + + GSList* it; + guint i = 0; + if (!config_dock_hide) + dock_get_area(&client_rects[i++]); + for (it = potential_overlap_clients; it != NULL; it = g_slist_next(it)) { + ObClient* potential_overlap_client = (ObClient*)it->data; + client_rects[i] = potential_overlap_client->frame->area; + i += 1; + } + g_slist_free(potential_overlap_clients); - *x = px - client->area.width / 2 - client->frame->size.left; -// *x = MIN(MAX(*x, l), r); - *y = py - client->area.height / 2 - client->frame->size.top; -// *y = MIN(MAX(*y, t), b); + Point result; + place_overlap_find_least_placement(client_rects, n_client_rects, head, + &frame_size, &result); + *x = result.x; + *y = result.y; return TRUE; } -static gboolean place_transient(ObClient *client, gint *x, gint *y) +static gboolean should_set_client_position(ObClient *client, + ObAppSettings *settings) { - if (client->transient_for) { - if (client->transient_for != OB_TRAN_GROUP) { - ObClient *c = client; - ObClient *p = client->transient_for; - *x = (p->frame->area.width - c->frame->area.width) / 2 + - p->frame->area.x; - *y = (p->frame->area.height - c->frame->area.height) / 2 + - p->frame->area.y; - return TRUE; - } else { - GSList *it; - gboolean first = TRUE; - gint l, r, t, b; - for (it = client->group->members; it; it = g_slist_next(it)) { - ObClient *m = it->data; - if (!(m == client || m->transient_for)) { - if (first) { - l = RECT_LEFT(m->frame->area); - t = RECT_TOP(m->frame->area); - r = RECT_RIGHT(m->frame->area); - b = RECT_BOTTOM(m->frame->area); - first = FALSE; - } else { - l = MIN(l, RECT_LEFT(m->frame->area)); - t = MIN(t, RECT_TOP(m->frame->area)); - r = MAX(r, RECT_RIGHT(m->frame->area)); - b = MAX(b, RECT_BOTTOM(m->frame->area)); - } - } - } - if (!first) { - *x = ((r + 1 - l) - client->frame->area.width) / 2 + l; - *y = ((b + 1 - t) - client->frame->area.height) / 2 + t; - return TRUE; - } - } - } - return FALSE; + gboolean has_position = settings && settings->pos_given; + gboolean has_forced_position = has_position && settings->pos_force; + + gboolean user_positioned = client->positioned & USPosition; + if (user_positioned && !has_forced_position) + return FALSE; + + gboolean program_positioned = client->positioned & PPosition; + if (program_positioned && !has_position) + return FALSE; + + return TRUE; } -void place_client(ObClient *client, gint *x, gint *y) +gboolean place_client(ObClient *client, gboolean client_to_be_foregrounded, + Rect* client_area, ObAppSettings *settings) { - gint rude = 1; - if (client->positioned) - rude = 0; - if (place_transient(client, x, y) || - ((config_place_policy == OB_PLACE_POLICY_MOUSE) ? - place_under_mouse(client, x, y) : - place_smart(client, x, y, SMART_FULL) || - place_smart(client, x, y, SMART_GROUP) || - place_smart(client, x, y, SMART_FOCUSED) || - place_random(client, x, y))) - { - /* make sure the window is visible. */ - client_find_onscreen(client, x, y, - client->frame->area.width, - client->frame->area.height, - /* non-normal clients has less rules, and - windows that are being restored from a session - do also. we can assume you want it back where - you saved it */ - client_normal(client) && !client->session); - /* get where the client should be */ - frame_frame_gravity(client->frame, x, y); - } else - g_assert_not_reached(); /* the last one better succeed */ + gboolean ret; + Rect *monitor_area; + int *x, *y, *w, *h; + Size frame_size; + + monitor_area = choose_monitor(client, client_to_be_foregrounded, settings); + + w = &client_area->width; + h = &client_area->height; + place_per_app_setting_size(client, monitor_area, w, h, settings); + + if (!should_set_client_position(client, settings)) + return FALSE; + + x = &client_area->x; + y = &client_area->y; + + SIZE_SET(frame_size, + *w + client->frame->size.left + client->frame->size.right, + *h + client->frame->size.top + client->frame->size.bottom); + + ret = + place_per_app_setting_position(client, monitor_area, x, y, settings, + frame_size) || + place_transient_splash(client, monitor_area, x, y, frame_size) || + place_under_mouse(client, x, y, frame_size) || + place_least_overlap(client, monitor_area, x, y, frame_size); + g_assert(ret); + + g_slice_free(Rect, monitor_area); + + /* get where the client should be */ + frame_frame_gravity(client->frame, x, y); + return TRUE; }