<table><tr><td style="">cblack requested changes to this revision.<br />cblack 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/D28114">View Revision</a></tr></table><br /><div><div><p>The 16px icons should probably drop the phone, as the distinguishing elements between them are so small as to impair the ease of telling the icons apart. Legibility should be prioritised over metaphor.</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/D28114">https://phabricator.kde.org/D28114</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, cblack<br /><strong>Cc: </strong>cblack, jbbgameich, bshah, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>