From a44b3a2daffa52640a2be20b904156abf1c585e5 Mon Sep 17 00:00:00 2001 From: Uli Schlachter Date: Sun, 30 Mar 2014 14:58:03 +0200 Subject: [PATCH] Don't set a background-pixel for our client frame windows Previously we would get a short black flicker when a client closes. This happened because first the client's window would get hidden and only a short moment later would awesome react to this and close its own window. In the mean time, the X server filled the frame window with its background-pixel which was black. Just removing the background-pixel means we get the default value which is None. This means that the content will be left untouched and the client's window will be visible for a moment longer. Signed-off-by: Uli Schlachter --- objects/client.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/objects/client.c b/objects/client.c index e7d6097e0..347cee460 100644 --- a/objects/client.c +++ b/objects/client.c @@ -455,12 +455,10 @@ client_manage(xcb_window_t w, xcb_get_geometry_reply_t *wgeom, bool startup) xcb_create_window(globalconf.connection, globalconf.default_depth, c->frame_window, s->root, wgeom->x, wgeom->y, wgeom->width, wgeom->height, wgeom->border_width, XCB_COPY_FROM_PARENT, globalconf.visual->visual_id, - XCB_CW_BACK_PIXEL | XCB_CW_BORDER_PIXEL | XCB_CW_BIT_GRAVITY - | XCB_CW_WIN_GRAVITY | XCB_CW_OVERRIDE_REDIRECT | XCB_CW_EVENT_MASK - | XCB_CW_COLORMAP, + XCB_CW_BORDER_PIXEL | XCB_CW_BIT_GRAVITY | XCB_CW_WIN_GRAVITY + | XCB_CW_OVERRIDE_REDIRECT | XCB_CW_EVENT_MASK | XCB_CW_COLORMAP, (const uint32_t []) { - globalconf.screen->black_pixel, globalconf.screen->black_pixel, XCB_GRAVITY_NORTH_WEST, XCB_GRAVITY_NORTH_WEST,