<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/D27224">View Revision</a></tr></table><br /><div><div><p>The pixel alignment for all parts of the inner symbol is off. The style of the inner symbol is a bit too rounded for the breeze style as well. For instance, the podium should have sharper corners in order to look more like other breeze icons. I know it's not really easy to tell what can be rounded and what can't. That's something we need to nail down in the HIG. In general, use roundness where appropriate for a given shape, not as a general style. Objects that typically have sharp corners should generally be represented with sharp corners.</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/D27224">https://phabricator.kde.org/D27224</a></div></div><br /><div><strong>To: </strong>mbruchert, dkardarakos, VDG, ndavis<br /><strong>Cc: </strong>ndavis, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>