Commit f8a9bdfd authored by Nate Graham's avatar Nate Graham 🔩

[RFC] Reduce sidebar icon size/increase list information density

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}
After: {F6685009}

Reviewers: #vdg

Subscribers: filipf, ndavis, broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D19694
parent dfac582d
......@@ -137,7 +137,6 @@ Kirigami.ScrollablePage {
delegate: Kirigami.BasicListItem {
id: delegate
icon: model.decoration
iconSize: Kirigami.Units.iconSizes.medium
label: model.display
separatorVisible: false
Accessible.role: Accessible.ListItem
......
......@@ -136,7 +136,6 @@ Kirigami.ScrollablePage {
delegate: Kirigami.BasicListItem {
id: delegate
icon: model.decoration
iconSize: Kirigami.Units.iconSizes.medium
label: model.display
separatorVisible: false
onClicked: systemsettings.activeSubCategory = index
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment