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

List:       koffice
Subject:    Re: icons
From:       David Faure <David.Faure () insa-lyon ! fr>
Date:       1999-02-18 18:33:47
[Download RAW message or body]

On Wed, Feb 17, 1999 at 02:44:28AM +0000, Kevin Forge wrote:
> Reginald Stadlbauer wrote:
> > 
> > Actually this is no problem. You just have also to change the
> > sources of the apps. Because if a KOffice app doesn't find all
> > icons, it crashes 
> 
> You are going to fix this right ? :)

Can I suggest an easy fix ?

Now that almost all koffice apps use ICON() (i.e. KIconLoader, from
kdelibs), the fix would be to patch KIconLoader to return the
"unknown.xpm" pixmap if the one asked can't be found.

Is there any application out there that relies on KIconLoader::loadIcon
returning 0L when the icon can't be found ?

(In this case we could add another parameter, with a default value).

What do you think ?

-- 
 ____________________________________________________________________
|                                                                    |
|  David FAURE                                                       |
|  E-mail : David.Faure@insa-lyon.fr, faure@kde.org                  |
|  http://www.insa-lyon.fr/People/AEDI/dfaure/index.html             |
|____________________________________________________________________|

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