From: Mikael Magnusson <mikachu@comhem.se>
Date: Sun, 6 Feb 2005 01:43:37 +0000 (+0000)
Subject: try moving the move_onscreen with the rude into the place routine so it only gets... 
X-Git-Url: https://git.dogcows.com/gitweb?a=commitdiff_plain;h=52c345c3002fe7bb134d35cb5446fa2601ff192f;p=chaz%2Fopenbox

try moving the move_onscreen with the rude into the place routine so it only gets called if a window doesn't specify a position
---

diff --git a/openbox/client.c b/openbox/client.c
index 2b30bddf..03789466 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -344,16 +344,6 @@ void client_manage(Window window)
 
         place_client(self, &x, &y);
 
-           
-        /* make sure the window is visible. */
-        client_find_onscreen(self, &x, &y,
-                             self->frame->area.width,
-                             self->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(self) && !self->session);
         if (x != ox || y != oy) 	 
             client_move(self, x, y);
     }
@@ -628,7 +618,7 @@ gboolean client_find_onscreen(ObClient *self, gint *x, gint *y, gint w, gint h,
     /* This here doesn't let windows even a pixel outside the screen,
      * not applied to all windows. Not sure if it's going to stay at all.
      * I wonder if disabling this will break struts somehow? Let's find out. */
-    if (0 && rude) {
+    if (rude) {
         /* avoid the xinerama monitor divide while we're at it,
          * remember to fix the placement stuff to avoid it also and
          * then remove this XXX */
diff --git a/openbox/place.c b/openbox/place.c
index 6669a28e..86af6b9d 100644
--- a/openbox/place.c
+++ b/openbox/place.c
@@ -385,6 +385,15 @@ void place_client(ObClient *client, gint *x, gint *y)
          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