From 15ede69da8cc5f107b96a51af5ad8f76bb0424ce Mon Sep 17 00:00:00 2001 From: Peter Penz Date: Tue, 8 Apr 2008 16:03:02 +0000 Subject: [PATCH] don't apply a frame to image previews, if the preview is a very small image (e. g. an icon) CCMAIL: aseigo@kde.org svn path=/trunk/KDE/kdebase/apps/; revision=794783 --- src/iconmanager.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/iconmanager.cpp b/src/iconmanager.cpp index cfdf13a7c..b46328cea 100644 --- a/src/iconmanager.cpp +++ b/src/iconmanager.cpp @@ -294,8 +294,12 @@ void IconManager::applyCutItemEffect() bool IconManager::applyImageFrame(QPixmap& icon) { const QSize maxSize = m_view->iconSize(); - if ((maxSize.width() <= 24) || (maxSize.height() <= 24)) { - // the maximum size is too small for a frame + const bool applyFrame = (maxSize.width() > KIconLoader::SizeSmallMedium) && + (maxSize.height() > KIconLoader::SizeSmallMedium) && + ((icon.width() > KIconLoader::SizeLarge) || + (icon.height() > KIconLoader::SizeLarge)); + if (!applyFrame) { + // the maximum size or the image itself is too small for a frame return false; } -- 2.47.3