Hi Julian, On 16 June 2013 17:53, Julian Dammann <damma...@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