From 13dfc7fa2b883eaa88d6b4a03f17690d941008f8 Mon Sep 17 00:00:00 2001 From: Christoph Feck Date: Wed, 18 Jul 2012 01:22:55 +0200 Subject: [PATCH] Respect platform's icon sizes Use QStyle icon sizes instead of hard coding 16 or 32 pixels. Using styles that follow KDE settings (every KStyle, e.g. Oxygen) the icon sizes can be configured by the user in System Settings. The size for the icon/details view can be later changed by the user, overriding the defaults. The icon size for the places panel is not (yet?) configurable separately. (Not resolving the bug, see comment in review request) CCBUG: 301959 REVIEW: 105548 --- src/kitemviews/kstandarditemlistview.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/kitemviews/kstandarditemlistview.cpp b/src/kitemviews/kstandarditemlistview.cpp index f4d05dcf6..79eb86b89 100644 --- a/src/kitemviews/kstandarditemlistview.cpp +++ b/src/kitemviews/kstandarditemlistview.cpp @@ -141,9 +141,9 @@ void KStandardItemListView::onSupportsItemExpandingChanged(bool supportsExpandin void KStandardItemListView::polishEvent() { switch (m_itemLayout) { - case IconsLayout: applyDefaultStyleOption(KIconLoader::SizeMedium, 2, 4, 8); break; - case CompactLayout: applyDefaultStyleOption(KIconLoader::SizeSmall, 2, 8, 0); break; - case DetailsLayout: applyDefaultStyleOption(KIconLoader::SizeSmall, 2, 0, 0); break; + case IconsLayout: applyDefaultStyleOption(style()->pixelMetric(QStyle::PM_LargeIconSize), 2, 4, 8); break; + case CompactLayout: applyDefaultStyleOption(style()->pixelMetric(QStyle::PM_SmallIconSize), 2, 8, 0); break; + case DetailsLayout: applyDefaultStyleOption(style()->pixelMetric(QStyle::PM_SmallIconSize), 2, 0, 0); break; default: Q_ASSERT(false); break; } -- 2.47.3