[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: servicemenu patch
From:       Antonio Larrosa <antlarr () arrakis ! es>
Date:       2000-10-27 18:09:12
[Download RAW message or body]

Torsten Rahn escribi=F3:
> =

> Am Freitag 27 Oktober 2000 18:13 schrieb Matthias Elter:
> > On Friday 27 October 2000 16:01, Torsten Rahn wrote:
> > > Am Freitag 27 Oktober 2000 16:30 schrieb Matthias Elter:
> > > > > Hi
> > > >
> > > > KIconloader returns large icons when it can't find a small versio=
n even
> > > > if the KIcon::Small override flag is set. This makes the K-Menu l=
ook
> > > > ugly on some systems (COL for example ;-). ServiceMenu will now c=
heck
> > > > the size of the icons returned by KIconLoader and smoothScale 'em=
 if
> > > > required.
> > >
> > > Stop! Does it scale them in *ANY* case? i.e. if you have a 16x16-ve=
rsion
> > > available, does it scale this icon up to 48x48?
> > > If it does you didn't understand certain concepts concerning the
> > > iconloader ....
> >
> > Read the patch! It's about menus. There are no icons larger than 16x1=
6 in
> > menus. So I scale down when needed but not the other way round.
> =

> Ok, I talked to matthias and instead of breaking concepts by workaround=
s
> we'll simply replace "Fixed" by "Scaled" by Default in the desktop-file=
s.
> This will scale icons to the appropriate size whenever necessary.
> =


I'd prefer to apply the patch than making all the icons scalable.

If the icons are scalable some more structs are made internally
and only this will be fixed by making things a bit more complicated
when a simple patch is enough.

Greetings,

--
Antonio Larrosa Jimenez
KDE core developer
antlarr@arrakis.es        larrosa@kde.org
http://www.arrakis.es/~rlarrosa
KDE - The development framework of the future, today.

[prev in list] [next in list] [prev in thread] [next in thread]