<table><tr><td style="">ngraham added a comment.
</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/D25815">View Revision</a></tr></table><br /><div><div><p>Very nice! You even embedded the stylesheet. However there are a number of lines and corners that would look better aligned to grid lines or points. You don't have to always to this, but in general it's better to. Also the Breeze icon style generally has pointy corners rather than rounded ones.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25815">https://phabricator.kde.org/D25815</a></div></div><br /><div><strong>To: </strong>vinzenzv, VDG<br /><strong>Cc: </strong>ngraham, iasensio, kde-frameworks-devel, LeGast00n, fbampaloukas, GB_2, trickyricky26, michaelh, bruns<br /></div>