grab the pointer on the screen_support_win, causing all mouse events during a grab...

[chaz/openbox] / openbox / moveresize.c

diff --git a/openbox/moveresize.c b/openbox/moveresize.c
index 8928a995d75d7bb55935e2e71e33dca204f3073e..447869aa29435a56399042cc501f49ba6b7a3430 100644 (file)
--- a/openbox/moveresize.c
+++ b/openbox/moveresize.c
@@ -26,7 +26,7 @@ static guint button;
 static guint32 corner;
 static ObCorner lockcorner;
 
-static Popup *popup = NULL;
+static ObPopup *popup = NULL;
 
 static void client_dest(gpointer client)
 {
@@ -44,8 +44,11 @@ void moveresize_startup(gboolean reconfig)
 
 void moveresize_shutdown(gboolean reconfig)
 {
-    if (!reconfig)
+    if (!reconfig) {
+        if (moveresize_in_progress)
+            moveresize_end(FALSE);
         client_remove_destructor(client_dest);
+    }
 
     popup_free(popup);
     popup = NULL;
@@ -61,7 +64,7 @@ static void popup_coords(ObClient *c, char *format, int a, int b)
                    c->area.width / 2,
                    c->frame->area.y + c->frame->size.top +
                    c->area.height / 2);
-    popup_show(popup, text, NULL);
+    popup_show(popup, text);
     g_free(text);
 }
 
@@ -71,6 +74,9 @@ void moveresize_start(ObClient *c, int x, int y, guint b, guint32 cnr)
 
     g_assert(!moveresize_in_progress);
 
+    if (!c->frame->visible)
+        return;
+
     moveresize_client = c;
     start_cx = c->frame->area.x;
     start_cy = c->frame->area.y;
@@ -79,8 +85,8 @@ void moveresize_start(ObClient *c, int x, int y, guint b, guint32 cnr)
        increment instead of at 0, so you have to move half an increment
        either way instead of a full increment one and 1 px the other. and this
        is one large mother fucking comment. */
-    start_cw = c->area.width + (c->size_inc.width + 1) / 2;
-    start_ch = c->area.height + (c->size_inc.height + 1) / 2;
+    start_cw = c->area.width + c->size_inc.width / 2;
+    start_ch = c->area.height + c->size_inc.height / 2;
     start_x = x;
     start_y = y;
     corner = cnr;
@@ -168,7 +174,8 @@ static void do_move(gboolean resist)
     /* get where the client should be */
     frame_frame_gravity(moveresize_client->frame, &cur_x, &cur_y);
     client_configure(moveresize_client, OB_CORNER_TOPLEFT, cur_x, cur_y,
-                     start_cw, start_ch, TRUE, FALSE);
+                     moveresize_client->area.width,
+                     moveresize_client->area.height, TRUE, FALSE);
 }
 
 static void do_resize(gboolean resist)