<table><tr><td style="">ndavis requested changes to this revision.<br />ndavis added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18432">View Revision</a></tr></table><br /><div><div><p>Make the outline a solid circle behind the clock part instead of just an outline. The reason for this is that the edges between the outline and the clock look worse if the outline isn't a solid circle in the background. Notice how the left side has rougher edges: <a href="https://phabricator.kde.org/F6560806" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6560806: Screenshot_20190121_123932.png</a><br />
In general, it's a good idea to not make 2 shapes meet exactly at the edges due to issues with how SVGs are displayed. This isn't an Inkscape only either. It doesn't always need to be done that way, but it's especially necessary with shapes that have round edges.</p>

<p>I think it might be a good idea to make it look more like a clock, but I'm not going to object to keeping the current look.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18432">https://phabricator.kde.org/D18432</a></div></div><br /><div><strong>To: </strong>trickyricky26, VDG, ngraham, ndavis<br /><strong>Cc: </strong>ndavis, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>