<table><tr><td style="">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/D18797">View Revision</a></tr></table><br /><div><div><p>Because the default MainToolbar size is 22x22, at which size we have various monochrome versions of color icons. So for example it would display the ugly 22px monochrome version of the Fonts icon, which did not change its color due to a bug in the way this tooltip is generated (it was never intended to draw monochrome icons because it does not invoke their color-changing behavior).</p>

<p>By hardcoding a 24px version, KIconLoader correctly fails to find a 24px monochrome version of these icons and instead grabs the 32px color version and scales it down, which looks much better.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R124 System Settings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18797">https://phabricator.kde.org/D18797</a></div></div><br /><div><strong>To: </strong>ngraham, VDG<br /><strong>Cc: </strong>davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>