From kde-core-devel  Wed Sep 05 18:00:29 2007
From: Allen Winter <winter () kde ! org>
Date: Wed, 05 Sep 2007 18:00:29 +0000
To: kde-core-devel
Subject: Re: KPageWidget forces icons to 32x32
Message-Id: <200709051400.29606.winter () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=118901527715399

On Wednesday 05 September 2007 4:02:46 am David Faure wrote:
> 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.
> 

Ok, i'm committing that fix as I type this.
Thanks for the feedback David.