Re: [dev] Re - libsl/drw integration - patch

From: Anselm R Garbe <garbeam_AT_gmail.com>
Date: Sun, 16 Jun 2013 21:35:00 +0200

Hi Julian,

On 16 June 2013 17:53, Julian Dammann <dammannj_AT_googlemail.com> wrote:
> Hi,
>
> just looked at Anselm's commit 'finished libsl/drw integration'
> ( 5364697914fd4272fc1a6494b4fc522d2935427a ) and during merging a patch of mine
> I saw the no longer needed color enum, and the new definitions of thmnorm and
> thmsel.
> As I use extra border colors for floating and urgent windows, I made a patch to
> use an array to hold themes, so that themes can be added as desired.
> I removed the color enum, and replaced it with a Theme enum.
>
> Comments welcome, please see attached patch.

I like your approach, but haven't decided yet if I consider the term
'theme' final yet, as it might be a bit misleading.

If anyone has a better term for a color tuple for
foreground/background/border, please step forward ;)

Best regards,
Anselm
Received on Sun Jun 16 2013 - 21:35:00 CEST