<table><tr><td style="">ndavis 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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18797#406754" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18797#406754</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Next question: KInfoCenter's version of this tooltip currently hardcodes 32x32 for the size of the parent icon above it rather than using the <tt style="background: #ebebeb; font-size: 13px;">KIconLoader::Dialog</tt> size (which defaults to 32x32).. Should we do that here too so that there's never a case where the parent icon could be smaller than the child icon for the corner case of a user who has set the <tt style="background: #ebebeb; font-size: 13px;">KIconLoader::Dialog</tt> size to something smaller than its default?</p></div>
</blockquote>

<p>Is hardcoding a good idea? What if a new size was added to KIconLoader with a default size of 24px? Then we could reuse the same thing in Plasma's Kicker menu to make the icons more colorful and make the 32 and 48 px icons scale down 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>ndavis, davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>