From 96350151b91dbfd0dc00d3726d0e51e912134852 Mon Sep 17 00:00:00 2001 From: Nikos Ntarmos Date: Thu, 8 Nov 2007 11:31:37 +0100 Subject: [PATCH] Fix issue with multiple clients having focused border on same tag It so happens that when two clients are fired up one after the other on the same tag, they both get a 'focused'-type border. A bisect sequence showed that the culprit was commit 001f430. I think that it all boils down to client_manage just setting tag->client_sel and hoping for arrange(...) to do the Right Thing (TM). The attached patch uses focus(...) instead. --- client.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/client.c b/client.c index bc20e4ea..3c38d47d 100644 --- a/client.c +++ b/client.c @@ -262,7 +262,6 @@ void client_manage(Window w, XWindowAttributes *wa, awesome_config *awesomeconf) { int i; - Tag *tag; Client *c, *t = NULL; Window trans; Status rettrans; @@ -364,8 +363,7 @@ client_manage(Window w, XWindowAttributes *wa, awesome_config *awesomeconf) /* some windows require this */ XMoveResizeWindow(c->display, c->win, c->x, c->y, c->w, c->h); - if((tag = get_current_tag(awesomeconf->tags, awesomeconf->ntags))) - tag->client_sel = c; + focus(c, True, awesomeconf); /* rearrange to display new window */ arrange(awesomeconf);