On Sun, May 6, 2012 at 2:00 PM, Uli Schlachter <[email protected]> wrote: > On 06.05.2012 04:20, Anurag Priyam wrote: >> Uli, what's with the `colstr` variable? It doesn't seem to be used >> anywhere. Same for the `if c:get_type()` block. Doesn't look like >> anything is being done with the rgba values that it computes. I >> wonder if this (attached patch) is the right approach: > > This was added in commit 07bbe14d02. The problem was that whenever the wibox > needed to be drawn, the X server first filled that area with black and only [...]
Makes sense.
>> function set_bg(wibox, c)
[...]
>> + c = string_format("#%02x%02x%02x", r, g, b)
>> end
>> wibox.background_color = c
>> wibox.draw()
>
> This patch is wrong. wibox.background_color must always be a cairo pattern
> object and not a string. If it really gets set to "#000000", everytime a wibox
> gets redrawn, oocairo will throw a lua error about "cr:set_source() needs a
> Cairo pattern or surface object as argument 2".
Ok. I thought the computed rgb values had to be used.
> Could you send (to awesome-devel) two new patches? First one will be
[...]
Attached.
--
Anurag Priyam
0001-wibox-assume-a-default-background-and-foreground-col.patch
Description: Binary data
0002-wibox.set_bg-remove-historical-cruft.patch
Description: Binary data
