fix(lib.awful.taglist): multiple tag selection

[actionless]

multiple tag selection wasn't highlighting properly when *_occupied is set in theme table

[psychon]

That merge commit seems useless and I don't see why I should pull that.

For the actual issue: I don't understand what you are trying to fix. The commit message says that this is due to *_occupied, but uhm.... dunno.

What does this have to do with selecting multiple tags? Any ideas how the code can be simplified? Moving the check for t.selected around would also likely leave behind some dead code... at least there are other places checking for t.selected as well. I'm confused.

Also, sorry for my slow reaction to this.

[actionless]

For the actual issue: I don't understand what you are trying to fix. The commit message says that this is due to *_occupied, but uhm.... dunno.

to reproduce that:

1. set in theme.lua (which will be loaded by beautiful module) theme.taglist_font, theme.taglist_fg_occupied, theme.taglist_bg_occupied, theme.taglist_fg_empty, theme.taglist_bg_empty, theme.taglist_fg_focus, theme.taglist_bg_focus to different colors
2. select multiple tags (AFAIR, by default it's binded to modkey+ Control + {1,2,3...})

in that situation multiple selected tags are colored using theme.taglist_fg_occupied, theme.taglist_bg_occupied colors instead of theme.taglist_fg_focus, theme.taglist_bg_focus

[psychon]

Ok. I have to agree that the behavior is way better with your change.

Could you get rid of that additional merge commit? Or is it ok if I just cherry pick your commit?

[actionless]

i think cherry-picking should be ok

[psychon]

Cherry picked as commit 9390534