[Sugar-devel] [PATCH sugar-artwork] Zoom icons: Removing grey fill color

[Gary Martin]

Hi Daniel,

On 13 Jul 2012, at 16:00, S. Daniel Francis wrote:

> I made a new patch with similar changes, but for the table/cell icons,
> also mailed to this list.

Thanks for the patches! I've not had time to review them yet, design review on svg diff patches sent to the mail list is a petty slow work-flow for me to actually get to see the end result ;) But while you pretend to listen to elevator music, waiting for a review, I thought you might find this of interest. It's an svg gird of all the other svg's currently in sugar-artwork mainline, hover over them to see the file name. It works well in Safari and Chrome (but I think Firefox's SVG support is still a little broken with regard to the official spec supporting the imbedding of other images in an SVG):

	http://wiki.sugarlabs.org/images/e/e8/Sugar-artwork.svg

There's quite a lot of unused items in there, and some you see with grey are often intended to allow for the user fill and stroke entity colours (but pretty sure the one's you've patched so far were not intended for such user colour use). I'll clarify/confirm once I've had a chance to look over your patches in more detail.

Regards,
--Gary

> Cheers.
> 
> 2012/7/13 Gonzalo Odiard <gonzalo at laptop.org>:
>> Maybe Gary or Manuq can look at this issue?
>> 
>> Gonzalo
>> 
>> _______________________________________________
>> Sugar-devel mailing list
>> Sugar-devel at lists.sugarlabs.org
>> http://lists.sugarlabs.org/listinfo/sugar-devel
>>