<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham 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/D23300">View Revision</a></tr></table><br /><div><div><p>Interesting design! However having that triangular part jutting out of the side is pretty unusual, and not seen in any other MimeType icons. Maybe move that shape fully inside and copy the visual style of existing book-style icons, such as <tt style="background: #ebebeb; font-size: 13px;">application-pdf</tt> <tt style="background: #ebebeb; font-size: 13px;">application-epub+zip</tt> . Generally they have a sort of book-like appearance with a line on the left side for the book's spine.</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/D23300">https://phabricator.kde.org/D23300</a></div></div><br /><div><strong>To: </strong>yurchor, VDG, ngraham<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>