<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type"> <title></title> </head> <body bgcolor="#ffffff" text="#000000"> Martin Sourada a écrit : <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap=""> Good. Some comments: * the 16x16 is not symmetric, which makes it looks a little bit strange </pre> </blockquote> Should be fixed. <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap=""> * on the top of the front face in 256x256, the backgrounds leaks in, please fix that </pre> </blockquote> Fixed <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap=""> * I don't like the border in 256x256, could you do it traditionally with duplicating the shield, putting it below the actual shield and setting border to 1.5-2px with ~0.3 opacity and dark orange/brown colour? </pre> </blockquote> Choosing brown instead of orange seems to be a good fit. <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap=""> * Generally, I think the border should have been dark orange/brown, not metallic </pre> </blockquote> Done, set to brown. <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap=""> * Something feels a little bit strange about the shadow for me, could you try adding it more bellow the icon than behind it? </pre> </blockquote> Found that shadow was sandwiched between outline and inline drawing. <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap="">Some details might be good ;-) </pre> </blockquote> Added echo brand. <blockquote cite="mid:1221121671.3086.31.camel@pc-notebook" type="cite"> <pre wrap="">PS: IIRC, this is the last icon for System -> Administration in queue, so next we can move to <a class="moz-txt-link-freetext" href="https://fedorahosted.org/echo-icon-theme/wiki/ToDo/BaseSet">https://fedorahosted.org/echo-icon-theme/wiki/ToDo/BaseSet</a> </pre> </blockquote> Good hear. echo 0.3.3 should be released for both Fedora 8 and 9. Luya Reference: -------------- <a class="moz-txt-link-freetext" href="http://luya.fedorapeople.org/echo/status/system-root-password.svg">http://luya.fedorapeople.org/echo/status/system-root-password.svg</a> </body> </html>