much simpler and works just as well! client's reference position doesnt change just...

diff --git a/openbox/openbox.c b/openbox/openbox.c
index d4f8423219db65d4773af9f9f98e8f9488d44bea..b7d4f5cc57362995b50e5c899833158911dc2018 100644 (file)
--- a/openbox/openbox.c
+++ b/openbox/openbox.c
@@ -331,70 +331,14 @@ gint main(gint argc, gchar **argv)
                 for (it = client_list; it; it = g_list_next(it)) {
                     ObClient *c = it->data;
                     Strut oldsize, newsize;
-                    gint dx, dy;
-
-                    oldsize = c->frame->size;
-                    dx = dy = 0;
 
                     /* the new config can change the window's decorations */
                     client_setup_decor_and_functions(c, TRUE);
                     /* redraw the frames */
                     frame_adjust_area(c->frame, TRUE, TRUE, FALSE);
-
-                    /* move windows if their decorations are a new size, based
-                       on their gravity */
-                    newsize = c->frame->size;
-
-                    /* x coord */
-                    switch (c->gravity) {
-                    case NorthWestGravity:
-                    case WestGravity:
-                    case SouthWestGravity:
-                        break;
-                    case NorthGravity:
-                    case CenterGravity:
-                    case SouthGravity:
-                        dx = (newsize.left - oldsize.left +
-                              oldsize.right - newsize.right) / 2;
-                        break;
-                    case NorthEastGravity:
-                    case EastGravity:
-                    case SouthEastGravity:
-                        dx = -(newsize.left - oldsize.left +
-                               newsize.right - oldsize.right);
-
-                        break;
-                    case StaticGravity:
-                        dx = -(newsize.left - oldsize.left);
-                        break;
-                    }
-
-                    /* y coord */
-                    switch (c->gravity) {
-                    case NorthWestGravity:
-                    case NorthGravity:
-                    case NorthEastGravity:
-                        break;
-                    case WestGravity:
-                    case CenterGravity:
-                    case EastGravity:
-                        dy = (newsize.top - oldsize.top +
-                              oldsize.bottom - newsize.bottom) / 2;
-                        break;
-                    case SouthWestGravity:
-                    case SouthGravity:
-                    case SouthEastGravity:
-                        dy = -(newsize.top - oldsize.top +
-                               newsize.bottom - oldsize.bottom);
-
-                        break;
-                    case StaticGravity:
-                        dx = -(newsize.top - oldsize.top);
-                        break;
-                    }
-
-                    if (dx || dy)
-                        client_move(c, c->area.x + dx, c->area.y + dy);
+                    /* the decor sizes may have changed, so the windows may
+                       end up in new positions */
+                    client_reconfigure(c);
                 }
             }