Bug 67450 - Kontact does not load the correct icons for new mail, todo, event, etc... in the Kontact toolbar
Summary: Kontact does not load the correct icons for new mail, todo, event, etc... in ...
Status: RESOLVED FIXED
Alias: None
Product: kontact
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-06 22:45 UTC by Carlos Woelz
Modified: 2004-01-02 19:39 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Woelz 2003-11-06 22:45:05 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc 3.2.2 mdk
OS:          Linux

Simple bug. See above.
I mean the first icon on the right.
Comment 1 Carlos Woelz 2003-11-06 22:46:57 UTC
Left, not right.
Comment 2 Tobias Koenig 2003-11-21 19:30:25 UTC
Subject: kdelibs/kdeui

CVS commit by tokoe: 

Use iconSet when available in KToolBarPopupAction, otherwise the wrong
icons are loaded when actions from different parts (instances) are plugged in.

CCMAIL:67450-done@bugs.kde.org


  M +15 -9     kactionclasses.cpp   1.332


--- kdelibs/kdeui/kactionclasses.cpp  #1.331:1.332
@@ -1812,4 +1812,9 @@ int KToolBarPopupAction::plug( QWidget *
     int id_ = KAction::getToolButtonID();
 
+    if ( !iconSet().isNull() ) {
+        bar->insertButton( iconSet().pixmap(), id_, SIGNAL( clicked() ), this,
+                           SLOT( slotActivated() ), isEnabled(), plainText(),
+                           index );
+    } else {
     KInstance * instance;
     if ( m_parentCollection )
@@ -1821,4 +1826,5 @@ int KToolBarPopupAction::plug( QWidget *
                        SLOT( slotActivated() ), isEnabled(), plainText(),
                        index, instance );
+    }
 
     addContainer( bar, id_ );


Comment 3 Carlos Woelz 2003-12-15 03:36:50 UTC
I still have the general icon for new mail. The others seem fixed.
Comment 4 Volker Krause 2004-01-02 19:39:33 UTC
Subject: kdepim/kontact/plugins/kmail

CVS commit by vkrause: 

There is no "mail_new2" icon anymore.

CCMAIL: 67450-done@bugs.kde.org


  M +1 -1      kmail_plugin.cpp   1.26


--- kdepim/kontact/plugins/kmail/kmail_plugin.cpp  #1.25:1.26
@@ -48,5 +48,5 @@ KMailPlugin::KMailPlugin(Kontact::Core *
   setInstance( KMailPluginFactory::instance() );
 
-  insertNewAction( new KAction( i18n( "New Mail" ), BarIcon( "mail_new2" ),
+  insertNewAction( new KAction( i18n( "New Mail" ), BarIcon( "mail_new" ),
                                      0, this, SLOT( slotNewMail() ), actionCollection(),
                    "new_mail" ) );