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.
This commit is contained in:
parent
10c257d57a
commit
e79fdcb22e
5
tag.c
5
tag.c
|
@ -335,8 +335,13 @@ uicb_tag_view(int screen, char *arg)
|
||||||
if(arg)
|
if(arg)
|
||||||
tag_view(screen, atoi(arg) - 1);
|
tag_view(screen, atoi(arg) - 1);
|
||||||
else
|
else
|
||||||
|
{
|
||||||
for(tag = globalconf.screens[screen].tags; tag; tag = tag->next)
|
for(tag = globalconf.screens[screen].tags; tag; tag = tag->next)
|
||||||
tag->selected = True;
|
tag->selected = True;
|
||||||
|
saveawesomeprops(screen);
|
||||||
|
arrange(screen);
|
||||||
|
ewmh_update_net_current_desktop(get_phys_screen(screen));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/** View previously selected tags
|
/** View previously selected tags
|
||||||
|
|
Loading…
Reference in New Issue