D24683: [KWindowSystem] in icon() return realistic icon size

Marco Martin noreply at phabricator.kde.org
Wed Oct 16 09:34:49 BST 2019


mart added inline comments.

INLINE COMMENTS

> windowsystem.cpp:173
> +        iconWidth = 16;
> +    } else if (width < 40) {
> +        iconWidth = 32;

what's the logic behind the numbers 40, 56, 96 ans 192?
should at least be in a comment

REPOSITORY
  R130 Frameworks integration plugin using KWayland

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

To: meven, #kwin, davidedmundson
Cc: mart, zzag, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191016/85abbfb7/attachment.html>


More information about the Plasma-devel mailing list