[Ml-yokadi] renderTaskListRow is buggy
Aurélien Gâteau
aurelien.gateau
Mar 24 Fév 09:43:01 CET 2009
Sébastien Renard wrote:
> Le samedi 21 février 2009, Aurélien Gâteau a écrit :
>> Sébastien Renard wrote:
>>> Le jeudi 19 février 2009, Sébastien Renard a écrit :
>>>> Mmm... I came up with a little-over-engineered solution.
>>>>
>>>> Just a little ;-)
>>>>
>>>>> What do you
>>>>> think about it?
>>>> Quite complicated, but I agree about the method (using ljust and color
>>>> after), see my previous mail that I wrote this morning in the train
>>>> before seeing you answer.
>>> It will probably conflict with the patch I did this morning (just pushed)
>>> to convert creationDate to age. Feel free to overwrite it.
>>>
>>> Please commit you patch. Even if it's quite complicated, it solved the
>>> problem. We can work in a second time to simplify it (for ex. make a
>>> general "colorising" function according to value).
>> Committed, merged and pushed!
>
> Thanks !
>
>> The new column is quite wide. I am working on a more compact version.
>
> I rework the color stuff, see commit.
Looks good. But I reworked it a bit more :). I merged the two callables
(text and color) into one. It adds a few functions and classes, but the
Column definition is much more readable.
Plus d'informations sur la liste de diffusion Ml-yokadi