Color picker should be populated with current color

[Zach Shepherd]

When the color picker is opened for a task that already has a color assigned, the current color should show up as "picked".

[Zach Shepherd (eightball1989)]

The attached diff/patch seems to fix this problem for me, although it does ignore the "notag_only" variable (I couldn't figure out quite what this did) as well as the case where the number of selected tags is anything other than 1 (I wasn't sure how to select multiple tags or what the appropriate behavior would be (should it just take the color of the first selected tag?).

This was my first look at the gtg code, so feedback would be appreciated.

[Zach Shepherd (eightball1989)]

Looks like I didn't test well enough. That appears to break when a tag doesn't have a color set. I'll give it another shot when I get a chance.

[Zach Shepherd (eightball1989)]

This tests to make sure that the current color is not "None".

[Lionel Dricot (ploum-deactivatedaccount)]

thanks Zach, I will revew it ASAP. For the record, "notag_only" is the case where you select to display only tasks that don't have a tag. (second line in the sidebar).

[Lionel Dricot (ploum-deactivatedaccount)]

This has been commited, thanks for the patch :-)