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

List:       kde-core-devel
Subject:    Re: kdegraphics/pixie
From:       Geert Jansen <g.t.jansen () stud ! tue ! nl>
Date:       2000-03-26 19:59:48
[Download RAW message or body]

Cristian Tibirna wrote:
> On Sun, 26 Mar 2000, Geert Jansen wrote:
> 
> > Kurt, you're mixing the concept Group and Size. It's twisted but Small is
> > actually a _group_. What group does the second line have? User of Small?
> 
> Call it something else. Like "Menus" or "Lists" or "Menus and Lists".
> Small is not only confusing, but wrong IMHO.
> 
> > 
> > For User type icons, the iconloader cannot know what size of icon the
> > API user wants. If there are multiple variants of an icon there (implied
> > by the mini- prefix), the developer should load them with their different
> > names.
> > 
> 
> Huh? This is wrong, isn't it? Where goes then our will to make globally
> configurable the kind of icons whole KDE uses? Does this mean one will
> have to put hooks and selectors in each and every icon loading in apps and
> trap desktop selections/changes of icon categories?

Ahem, am I so unclear? ...

No, this is right. You should only use UserIcon() if you have an
application specific pixmap of which you have only one version. If you have
multiple versions, install them under share/icons/, and user DesktopIcon(),
KickerIcon(), i.e. everything but UserIcon(). This will load the icon
according to the user's preferences. 

UserIcon() is just an easy accessor for icons in the application specific
directories. It should not be used, but a lot of code still does this. No
theming is possible with it.

I feel like I said this three times already.. oh wait, I did! :-)

Greetings,
-- 
    Geert Jansen                       email: <g.t.jansen at stud.tue.nl>
    Phylosopher, Physicist,                    email: <jansen at kde.org>           
    KDE enthusiast                                 PGP key ID: 0xD2B5E7CE            

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