okay so that's why it didn't use move_onscreen

diff --git a/openbox/client.c b/openbox/client.c
index e37c1d241c22c66bbf969c49f2779ee736d5cc41..2b30bddf8686dad04f09a27e890e02c019b0f4cf 100644 (file)
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -344,14 +344,18 @@ void client_manage(Window window)
 
         place_client(self, &x, &y);
 
-        /* make sure the window is visible.
            
-           this is about the rude parameter:
-           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_move_onscreen(self, client_normal(self) && !self->session);
+        /* 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);
     }
 
     keyboard_grab_for_client(self, TRUE);