<table><tr><td style="">andreyorst 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/D8912">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p><a href="https://phabricator.kde.org/p/andreyorst/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@andreyorst</a>, if you're still around, we can review this patch and land it once necessary changes are made.</p></blockquote>

<p>Yeah, I'm here! I'd like to make desired changes myself.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Nice job on the 64 px icons! However, I noticed that the 16 and 22 px icons use a different style and are blurry. Perhaps you should try making the styles more similar to the 64 px style and see if you can move, resize or remove any details that don't line up with pixels.</p></blockquote>

<p>So the reason behind this was, that with outline style for 16 and 22 px icons all elements like buttons were too small and looked really bad. I've tried lot of variants and decided to go for filled icon in the same way how it is done for <tt style="background: #ebebeb; font-size: 13px;">network-server</tt> icon, which is using (sort of) outline for 64 px and solid for 22 px. However (the ticket is quite old) I've changed this somewhere during this year, and currently in my system all icons are using filled icons, since I feel that this fits more with original style of <tt style="background: #ebebeb; font-size: 13px;">applications-games</tt> icon (and most of game controllers are black too). I'll submit updated patch this evening.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I noticed that the 16 and 22 px icons use a different style and are blurry</p></blockquote>

<p>What screen resolution do you have? Rightnow it looks fine on my system with 1920x1080 screen. I'll check it at work with 2k screen and scaling turned off.</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/D8912">https://phabricator.kde.org/D8912</a></div></div><br /><div><strong>To: </strong>andreyorst, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, ndavis, michaelh, bruns<br /></div>