
> It used to do that, I removed that check because it prevented e.g.
> adding the location label or combo in Konqueror. Those actions don't
> have icons, but they don't need one either. The test is actually a bit
> more precise, it already skips plain KActions that have no icon,
> but shows KAction-derived actions that have an icon. See the
> isA("KAction") test in kedittoolbar.cpp:648.
>
> Anyway the real fix is to implement "letting the user choose an icon".
> I can see you volunteering from here ;)

Couldn't the guy who volunteers for this also volunteer for 
moving the whole "Show blablabla-toolbar" - Menu items into a listview which 
would be shown on a second tab under "Configure Toolbars". People really 
_don't_ reconfigure their toolbars that often. Actually most people only 
configure them once per app at maximum. Therefore it would be nice if all 
those menuitems wouldn't clutter the whole menu so much and would be 
displayed on a second tab on the dialog "Configure Toolbars" instead.
That dialog should also make it possbile to create/delete/modify toolbars.

Greetings,
Tackat

-- 
Dipl.-Phys. Torsten Rahn 
<tackat@suse.de>   <tackat@kde.org>   <tackat@tackat.org>

KDE - Conquer Your Desktop!
http://www.kde.org



