From kde-core-devel  Wed Sep 05 08:02:46 2007
From: David Faure <faure () kde ! org>
Date: Wed, 05 Sep 2007 08:02:46 +0000
To: kde-core-devel
Subject: Re: KPageWidget forces icons to 32x32
Message-Id: <200709051002.46977.faure () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=118897943202297

On Wednesday 05 September 2007, Jason 'vanRijn' Kasper wrote:
> I don't believe I ever saw any responses to my question.  The situation is 
> still the same, I believe.  The size of the pixmap is still being hard-coded 
> to 32x32:
> 
> .-(~/kde4/src/kdelibs/kdeui/paged)-
> `--> grep 32 *
> kpageview_p.cpp:  const QPixmap pixmap = icon.pixmap( 32, 32 );
> kpageview_p.cpp:  const QPixmap pixmap = icon.pixmap( 32, 32 );
> 
> I guess that for KPilot, it's not that big of a deal, and we certainly have 
> bigger problems than the size of our icons not being the same size that they 
> were in kde3.  If nobody else is complaining about it, I'm content to drop 
> the issue.  =:)

This should use KIconLoader::global()->currentSize(K3Icon::Dialog) instead.

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).