D18797: [Tooltips] Use 24px size for children to improve display, just like KInfoCenter

[Noah Davis]

ndavis added a comment.


  In D18797#406754 <https://phabricator.kde.org/D18797#406754>, @ngraham wrote:
  
  > Next question: KInfoCenter's version of this tooltip currently hardcodes 32x32 for the size of the parent icon above it rather than using the `KIconLoader::Dialog` 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 `KIconLoader::Dialog` size to something smaller than its default?
  
  
  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.

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D18797

To: ngraham, #vdg
Cc: ndavis, davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190206/c0b0e390/attachment.html>