X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fplace.c;h=1ac295c0779b2758a27ae93230be1536758e3996;hb=55d2916c1e24e021d9b9692d2373dc4afff4c5c2;hp=ff45a4dccb858673a98359023a6ec170119fe613;hpb=29da19bd858d0e97dc90259d242f0ee26131b9a6;p=chaz%2Fopenbox

diff --git a/openbox/place.c b/openbox/place.c
index ff45a4dc..1ac295c0 100644
--- a/openbox/place.c
+++ b/openbox/place.c
@@ -2,7 +2,7 @@
 
    place.c for the Openbox window manager
    Copyright (c) 2006        Mikael Magnusson
-   Copyright (c) 2003        Ben Jansens
+   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
@@ -24,18 +24,52 @@
 #include "focus.h"
 #include "config.h"
 
-static Rect *pick_head(ObClient *c)
+static void add_choice(guint *choice, guint mychoice)
 {
-    Rect *area = NULL;
-    gint i;
+    guint i;
+    for (i = 0; i < screen_num_monitors; ++i) {
+        if (choice[i] == mychoice)
+            return;
+        else if (choice[i] == screen_num_monitors) {
+            choice[i] = mychoice;
+            return;
+        }
+    }
+}
+
+static Rect *pick_pointer_head(ObClient *c)
+{
+    guint i;
+    gint px, py;
+
+    screen_pointer_pos(&px, &py);
+     
+    for (i = 0; i < screen_num_monitors; ++i) {  
+        if (RECT_CONTAINS(*screen_physical_area_monitor(i), px, py)) {
+            return screen_area_monitor(c->desktop, i);
+        }
+    }
+    g_assert_not_reached();
+}
+
+static Rect **pick_head(ObClient *c)
+{
+    Rect **area;
+    guint *choice;
+    guint i;
+    gint px, py;
+
+    area = g_new(Rect*, screen_num_monitors);
+    choice = g_new(guint, screen_num_monitors);
+    for (i = 0; i < screen_num_monitors; ++i)
+        choice[i] = screen_num_monitors; /* make them all invalid to start */
 
     /* 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));
+        add_choice(choice, client_monitor(c->transient_for));
     }
 
-    /* more than one guy in his group (more than just him) */
+    /* more than one window in its group (more than just this window) */
     if (client_has_group_siblings(c)) {
         GSList *it;
 
@@ -45,33 +79,34 @@ static Rect *pick_head(ObClient *c)
             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));
+                add_choice(choice, client_monitor(it->data));
         }
 
         /* 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));
+                add_choice(choice, client_monitor(it->data));
         }
     }
 
+    if (focus_client)
+        add_choice(choice, client_monitor(focus_client));
+
     screen_pointer_pos(&px, &py);
 
-    for (i = 0; i < screen_num_monitors; i++) {
-        area = screen_area_monitor(client->desktop, i);
-        if (RECT_CONTAINS(*area, px, py))
+    for (i = 0; i < screen_num_monitors; i++)
+        if (RECT_CONTAINS(*screen_physical_area_monitor(i), px, py)) {
+            add_choice(choice, i);
             break;
-    }
-    if (i == screen_num_monitors)
-        area = screen_area_monitor(client->desktop, 0);    
+        }
 
-    /* Last resort */
-    if (!area)
-        area = screen_area_monitor(client->desktop,
-                                   g_random_int_range(0, screen_num_monitors));
+    /* add any leftover choices */
+    for (i = 0; i < screen_num_monitors; ++i)
+        add_choice(choice, i);
+
+    for (i = 0; i < screen_num_monitors; ++i)
+        area[i] = screen_area_monitor(c->desktop, choice[i]);
 
     return area;
 }
@@ -79,19 +114,23 @@ static Rect *pick_head(ObClient *c)
 static gboolean place_random(ObClient *client, gint *x, gint *y)
 {
     gint l, r, t, b;
-    Rect *area;
+    Rect **areas;
+    guint i;
 
-    area = pick_head(client);
+    areas = pick_head(client);
+    i = g_random_int_range(0, screen_num_monitors);
 
-    l = area->x;
-    t = area->y;
-    r = area->x + area->width - client->frame->area.width;
-    b = area->y + area->height - client->frame->area.height;
+    l = areas[i]->x;
+    t = areas[i]->y;
+    r = areas[i]->x + areas[i]->width - client->frame->area.width;
+    b = areas[i]->y + areas[i]->height - client->frame->area.height;
 
     if (r > l) *x = g_random_int_range(l, r + 1);
-    else       *x = 0;
+    else       *x = areas[i]->x;
     if (b > t) *y = g_random_int_range(t, b + 1);
-    else       *y = 0;
+    else       *y = areas[i]->y;
+
+    g_free(areas);
 
     return TRUE;
 }
@@ -220,40 +259,47 @@ typedef enum
 static gboolean place_smart(ObClient *client, gint *x, gint *y,
                             ObSmartType type)
 {
-    guint i;
     gboolean ret = FALSE;
     GSList *spaces = NULL, *sit;
     GList *it;
+    Rect **areas;
+    guint i;
 
-    spaces = area_add(spaces, pick_head(client));
+    if (type == SMART_GROUP) {
+        /* has to be more than me in the group */
+        if (!client_has_group_siblings(client))
+            return FALSE;
+    }
+
+    areas = pick_head(client);
+
+    for (i = 0; i < screen_num_monitors; ++i) {
+        spaces = area_add(spaces, areas[i]);
 
-    /* stay out from under windows in higher layers */
-    for (it = stacking_list; it; it = g_list_next(it)) {
-        ObClient *c;
+        /* stay out from under windows in higher layers */
+        for (it = stacking_list; it; it = g_list_next(it)) {
+            ObClient *c;
 
-        if (WINDOW_IS_CLIENT(it->data)) {
-            c = it->data;
-            if (c->fullscreen)
+            if (WINDOW_IS_CLIENT(it->data)) {
+                c = it->data;
+                if (c->fullscreen)
+                    continue;
+            } else
                 continue;
-        } else
-            continue;
 
-        if (c->layer > client->layer) {
-            if (!SMART_IGNORE(client, c))
-                spaces = area_remove(spaces, &c->frame->area);
-        } else
-            break;
-    }
+            if (c->layer > client->layer) {
+                if (!SMART_IGNORE(client, c))
+                    spaces = area_remove(spaces, &c->frame->area);
+            } else
+                break;
+        }
 
-    if (client->type == OB_CLIENT_TYPE_NORMAL) {
         if (type == SMART_FULL || type == SMART_FOCUSED) {
             gboolean found_foc = FALSE, stop = FALSE;
             ObClient *foc;
-            GList *list;
 
-            list = focus_order[client->desktop == DESKTOP_ALL ?
-                               screen_desktop : client->desktop];
-            foc = list ? list->data : NULL;
+            foc = focus_order_find_first(client->desktop == DESKTOP_ALL ?
+                                         screen_desktop : client->desktop);
 
             for (; it && !stop; it = g_list_next(it)) {
                 ObClient *c;
@@ -277,10 +323,6 @@ static gboolean place_smart(ObClient *client, gint *x, gint *y,
                     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))
@@ -288,44 +330,47 @@ static gboolean place_smart(ObClient *client, gint *x, gint *y,
             }
         } else
             g_assert_not_reached();
-    }
 
-    spaces = g_slist_sort_with_data(spaces, area_cmp, client);
+        spaces = g_slist_sort_with_data(spaces, area_cmp, client);
 
-    for (sit = spaces; sit; sit = g_slist_next(sit)) {
-        Rect *r = sit->data;
+        for (sit = spaces; sit; sit = g_slist_next(sit)) {
+            Rect *r = sit->data;
 
-        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;
-                } else {
-                    *x = r->x;
-                    *y = r->y;
+            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;
+                    } else {
+                        *x = r->x;
+                        *y = r->y;
+                    }
                 }
             }
-        }
 
-        g_free(r);
+            g_free(r);
+        }
+        g_slist_free(spaces);
+        spaces = NULL;
     }
-    g_slist_free(spaces);
+
+    g_free(areas);
 
     return ret;
 }
 
 static gboolean place_under_mouse(ObClient *client, gint *x, gint *y)
 {
-    guint i;
     gint l, r, t, b;
     gint px, py;
     Rect *area;
 
-    area = pick_head(client);
+    area = pick_pointer_head(client);
+    screen_pointer_pos(&px, &py);
 
     l = area->x;
     t = area->y;
@@ -343,7 +388,6 @@ static gboolean place_under_mouse(ObClient *client, gint *x, gint *y)
 static gboolean place_per_app_setting(ObClient *client, gint *x, gint *y,
                                       ObAppSettings *settings)
 {
-    gint px, py, i;
     Rect *screen;
 
     if (!settings || (settings && !settings->pos_given))
@@ -351,7 +395,7 @@ static gboolean place_per_app_setting(ObClient *client, gint *x, gint *y,
 
     /* Find which head the pointer is on */
     if (settings->head == -1)
-        screen = pick_head(client);
+        screen = pick_pointer_head(client);
     else
         screen = screen_area_monitor(client->desktop, settings->head);
 
@@ -374,18 +418,21 @@ static gboolean place_transient(ObClient *client, gint *x, gint *y)
         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;
+
+            if (client_normal(p)) {
+                *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 (!(m == client || m->transient_for) && client_normal(m)) {
                     if (first) {
                         l = RECT_LEFT(m->frame->area);
                         t = RECT_TOP(m->frame->area);
@@ -407,6 +454,19 @@ static gboolean place_transient(ObClient *client, gint *x, gint *y)
             }
         }
     }
+
+    if (client->transient) {
+        Rect **areas;
+
+        areas = pick_head(client);
+
+        *x = (areas[0]->width - client->frame->area.width) / 2 + areas[0]->x;
+        *y = (areas[0]->height - client->frame->area.height) / 2 + areas[0]->y;
+
+        g_free(areas);
+        return TRUE;
+    }
+
     return FALSE;
 }
 
@@ -429,6 +489,7 @@ gboolean place_client(ObClient *client, gint *x, gint *y,
          place_random(client, x, y))))
         g_assert_not_reached(); /* the last one better succeed */
     /* get where the client should be */
-    frame_frame_gravity(client->frame, x, y);
+    frame_frame_gravity(client->frame, x, y,
+                        client->area.width, client->area.height);
     return ret;
 }