ngraham created this revision. ngraham added a reviewer: VDG. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ngraham requested review of this revision.
REVISION SUMMARY Ever since increasing the list item size to accommodate 32px icons, System Settings' sidebar has felt somehow //wrong// to me. Though the icons are indeed pretty, and the problem with small monochrome icons has been resolved, the information density is now quite low. Since the list is very long, the amount of scrolliness is increased and there's a low of whitespace. This RFC patch explores the impact of reducing the icon size down to the default value for Kirigami list items. The information density increases and the whole thing feels better to me. On the other hand, the icons don't look as good because they're scaled to fit in a 22px size. Also, touch-friendliness is reduced. I'm not married to this idea but I thought I'd throw it out there. Comments are welcome. TEST PLAN Before: F6685010: Before.png <https://phabricator.kde.org/F6685010> After: F6685009: After.png <https://phabricator.kde.org/F6685009> REPOSITORY R124 System Settings BRANCH reduce-sidebar-icon-size (branched from master) REVISION DETAIL https://phabricator.kde.org/D19694 AFFECTED FILES sidebar/package/contents/ui/CategoriesPage.qml sidebar/package/contents/ui/SubCategoryPage.qml To: ngraham, #vdg Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
