On Wednesday 13 February 2002 22:18, Scott Wheeler wrote:
> On Wednesday 13 February 2002 12:50 pm, David Faure wrote:
> 
> > The value of m_iconSize has to remain IMHO.
> > The API, though, could be fixed, so that it returns the actual size, i.e.
> > iconSize() would query KIconTheme::currentSize (IIRC) to get the correct
> > size. However one would need to check if any KDE app uses iconSize()
> > currently, to see if it relies on it being 0 if default - could you check
> > that?
> 
> The only thing that I found (I greped my entire KDE dir structure) that uses 
> this is in kanimwidget.cpp:
> ===
>   if (parent()->inherits( "KToolBar" ))
>     d->size = ((KToolBar*)parent())->iconSize();
>   if (!d->size)
>      d->size = KGlobal::iconLoader()->currentSize(KIcon::MainToolbar);
> ===
> As you can see, it has a workaround for this in it that wouldn't be needed 
> with the suggested fix.  

Actually it should remain, just in case this isn't used in a KToolBar.
Better have a fallback in this unlikely case.

> I understand what you're saying about the internals of KToolBar.  I'm still 
> pretty new at hacking on the libs that I'm learning as I go.  Aside from 
> staying consistent with other implementations it didn't seem very 
> straightforward.

Isn't it ?
Suggested patch, posted here for review:

Index: ktoolbar.cpp
===================================================================
RCS file: /home/kde/kdelibs/kdeui/ktoolbar.cpp,v
retrieving revision 1.306
diff -u -p -r1.306 ktoolbar.cpp
--- ktoolbar.cpp        2002/02/09 00:01:24     1.306
+++ ktoolbar.cpp        2002/02/13 22:39:24
@@ -926,6 +926,8 @@ void KToolBar::setIconSize(int size, boo

 int KToolBar::iconSize() const
 {
+    if ( !d->m_iconSize ) // default value?
+        return KGlobal::iconLoader()->currentSize(KIcon::MainToolbar);
     return d->m_iconSize;
 }

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://people.mandrakesoft.com/~david, http://www.konqueror.org
KDE 3.0: Konquering the Desktops