Bugfix: Don’t use the minimum amount of pixels for dock clients which don’t set size...

diff --git a/src/manage.c b/src/manage.c
index e7b7c5f6920ac04e06b26da3f4e6b39061d16a59..e84c63453c70c218a2dc04dd40de6a46b86b3f6d 100644 (file)
--- a/src/manage.c
+++ b/src/manage.c
@@ -137,6 +137,7 @@ void reparent_window(xcb_connection_t *conn, xcb_window_t child,
                                   utf8_title_cookie, title_cookie, class_cookie;
         uint32_t mask = 0;
         uint32_t values[3];
+        uint16_t original_height = height;
 
         /* We are interested in property changes */
         mask = XCB_CW_EVENT_MASK;
@@ -276,13 +277,13 @@ void reparent_window(xcb_connection_t *conn, xcb_window_t child,
                            TODO: bars at the top */
                         new->desired_height = strut[3];
                         if (new->desired_height == 0) {
-                                LOG("Client wanted to be 0 pixels high, using the window's height (%d)\n", height);
-                                new->desired_height = height;
+                                LOG("Client wanted to be 0 pixels high, using the window's height (%d)\n", original_height);
+                                new->desired_height = original_height;
                         }
                         LOG("the client wants to be %d pixels high\n", new->desired_height);
                 } else {
-                        LOG("The client didn't specify space to reserve at the screen edge, using its height (%d)\n", height);
-                        new->desired_height = height;
+                        LOG("The client didn't specify space to reserve at the screen edge, using its height (%d)\n", original_height);
+                        new->desired_height = original_height;
                 }
         } else {
                 /* If it’s not a dock, we can check on which workspace we should put it. */