Calculate colors using the client's visual and colormap

As cwm was using the screen's default visual and colormap to draw all
client borders, borders for windows that had a depth of 32 bits were not
rendered correctly. The same happened with text in the popup menus which
were recently changed to be drawn in the context of the client.

This commit introduces a Visual reference for each client, and allocates
all potential colors for a client's specific visual and colormap in the
client_ctx struct. These colors are then used to draw client borders and
popup menus.

Additionally, since we touch the drawing code anyway, borders are
reintroduced to the popup menus for better contrast.

1 files changed, 5 insertions, 5 deletions

diff --git a/screen.c b/screen.c
index afd294c..0135449 100644
--- a/screen.c
+++ b/screen.c
@@ -265,13 +265,13 @@ screen_assert_clients_within(struct screen_ctx *sc)
 }
 
 void
-screen_prop_win_create(struct screen_ctx *sc, Window win)
+screen_prop_win_create(struct screen_ctx *sc, struct client_ctx *cc)
 {
-	sc->prop.win = XCreateSimpleWindow(X_Dpy, win, 0, 0, 1, 1, 0,
-	    sc->xftcolor[CWM_COLOR_MENU_BG].pixel,
-	    sc->xftcolor[CWM_COLOR_MENU_BG].pixel);
+	sc->prop.win = XCreateSimpleWindow(X_Dpy, cc->win, 0, 0, 1, 1, Conf.bwidth,
+	    cc->xftcolor[CWM_COLOR_MENU_FG].pixel,
+	    cc->xftcolor[CWM_COLOR_MENU_BG].pixel);
 	sc->prop.xftdraw = XftDrawCreate(X_Dpy, sc->prop.win,
-	    sc->visual, sc->colormap);
+	    cc->visual, cc->colormap);
 
 	XMapWindow(X_Dpy, sc->prop.win);
 }