<table><tr><td style="">ngraham requested changes to this revision.<br />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/D16815">View Revision</a></tr></table><br /><div><div><p>I know this is a bugfix, but...</p>

<p>IMHO it's really bad form to design a Breeze version of an app icon that has a totally different visual style from the original--especially if it completely eliminates all the branding elements from the original. This icon should be a Breezey version of the original icon, not something totally different that erases the Sublime Text brand. This is obviously not your fault, but I'd like to see the icon improved before we fix this bug, or else IMHO it will actually be a visual regression for Sublime Text users.</p>

<p>Really, the original mostly looks fine. A Breeze version should just harmonize the size with other square icons, reduce or eliminate the rounded corners, and maybe make some other very subtle changes. If we want to keep the parts of the current icon that depict a tabbed view, we could replace the lines of text with the orange S logo, or overlay it on top of them or something. I'm not sure if the tabbed view iconography is even necessary though.</p>

<p>All of this could be done in the same patch if you have icon design skills and want to tackle that, or in a separate one if not.</p>

<p>Oh, and thanks for the patch, and thanks for using <tt style="background: #ebebeb; font-size: 13px;">arc</tt> to submit it! :)</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/D16815">https://phabricator.kde.org/D16815</a></div></div><br /><div><strong>To: </strong>trickyricky26, VDG, ndavis, ngraham<br /><strong>Cc: </strong>ngraham, ndavis, kde-frameworks-devel, michaelh, bruns<br /></div>