trmdi added a comment.
On some small screens e.g <= 1024x768, `units.iconSizes.large` is a bit too big, it may look better with `units.iconSizes.medium` > property int iconSize: smallScreen ? units.iconSizes.large : units.iconSizes.huge Should I change it in this patch or create another one? Do we need to support <= 1024x768 screens nowadays ? Before: F6628578: Screenshot_20190221_225334.png <https://phabricator.kde.org/F6628578> After: F6628581: Screenshot_20190222_014900.png <https://phabricator.kde.org/F6628581> REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D19096 To: trmdi, ngraham, hein, #plasma, #vdg Cc: abetts, ndavis, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
