From e79fdcb22e9b7ec70ec223e13e798d1bcf4d56b3 Mon Sep 17 00:00:00 2001 From: Nikos Ntarmos Date: Thu, 3 Jan 2008 01:40:12 +0200 Subject: [PATCH] uicb_tag_view not working correctly with null argument uicb_tag_view(...) hasn't been working correctly since tag_view(...) came into existence. The problem is that saveawesomeprops(...), arrange(...) and ewmh_update_net_current_desktop(...) need to be called for a NULL argument as well, but these were moved to tag_view(...). We can either add them back albeit only for the arg=NULL case, or move these calls back into uicb_tag_view(...) and copy them over to ewmh_process_client_message(...) and to all possible future users of tag_view(...). The attached patch does the former. --- tag.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tag.c b/tag.c index 4219c2e6d..9fce6c956 100644 --- a/tag.c +++ b/tag.c @@ -335,8 +335,13 @@ uicb_tag_view(int screen, char *arg) if(arg) tag_view(screen, atoi(arg) - 1); else + { for(tag = globalconf.screens[screen].tags; tag; tag = tag->next) tag->selected = True; + saveawesomeprops(screen); + arrange(screen); + ewmh_update_net_current_desktop(get_phys_screen(screen)); + } } /** View previously selected tags