<table><tr><td style="">kmaterka created this revision.<br />kmaterka added reviewers: davidedmundson, Plasma, Plasma: Workspaces.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />kmaterka requested review of this revision.
</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/D24531">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Do not scale up small icons to avoid quality detoriation. For small icons scaling should be done in SystemTray only. Currently XembedSNIProxy scales to 32x32, then SystemTray (usually) scales it down to 24x24.</p>

<p>BUG: 366047</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I've run few applications with small icons:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">keepassx (22x22)</li>
<li class="remarkup-list-item">liferea (16x16)</li>
<li class="remarkup-list-item">tuxguitar (16x16)</li>
</ul>

<p>It looks much better without scaling in xemebdsniproxy.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R131 Plasma Wallpapers</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24531">https://phabricator.kde.org/D24531</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>xembed-sni-proxy/sniproxy.cpp</div></div></div><br /><div><strong>To: </strong>kmaterka, davidedmundson, Plasma, Plasma: Workspaces<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>