← Back to team overview

gtg-contributors team mailing list archive

Re: tag-bg-color patch

 

I also pushed code changes to launchpad. still not working, but I'll look
at it tomorrow morning if I have time, otherwise after Flourish! is over on
Saturday (most likely outcome).

Meg

2012/3/29 meg ford <meg387@xxxxxxxxx>

> Hi,
> I sent in the proposal. Please give feedback. I'll finish up the bug fix
> as soon as possible.
> Thanks!
> Meg
>
>
> 2012/3/29 meg ford <meg387@xxxxxxxxx>
>
>> + I'll push the new version of the code as soon as I get a chance so you
>> can take a look at it. Thanks for the help!
>>
>>
>> 2012/3/29 meg ford <meg387@xxxxxxxxx>
>>
>>> Hi,
>>>
>>> 2012/3/29 Izidor Matušov <izidor.matusov@xxxxxxxxx>
>>>
>>>> It is about really old API calls.
>>>>
>>>> If you replace lines 355 - 359 with this code:
>>>>
>>>> t = self.req.get_tag(tag)
>>>> if t is None:
>>>>  pass # something bad has happened and it should be handled
>>>> tag_color = t.get_attribute('color')
>>>> if tag_color is None:
>>>>  tag_color = <default color>
>>>>
>>>
>>> I tried it out, but it doesn't work yet. I think that what I'm doing is
>>> taking the color that has been assigned to the tag shapes in
>>> on_colorchooser_activate, and assigning it instead of the default color
>>> when it has been set, and keeping it 'default' when it hasn't. I think the
>>> problem is that the highlight (default) color doesn't have a setter (maybe
>>> the color is hardcoded in?).
>>>
>>> Does this sound at all correct? I'm getting this debug message:
>>> Traceback (most recent call last):
>>>   File "/home/meg/gtg/trunk/GTG/gtk/browser/browser.py", line 729, in
>>> on_colorchooser_activate
>>>     t.set_attribute("color", strcolor)
>>> AttributeError: 'str' object has no attribute 'set_attribute'
>>>
>>>
>>> Meg
>>>
>>>>
>>>>
>>>> That could should be equivalent. I haven't tested that code, writing
>>>> that just top of my head.
>>>>
>>>> Izidor
>>>>
>>>
>>>
>>
>

Follow ups

References