66740 – Add icons for context menu on address in readerwin

Version: (using KDE Devel)
Installed from: Compiled sources
Add icons for actions:
New Message To... (mail_new)
Reply To... (mail_reply)
Forward To... (mail_forward)
in RMB menu when clicking on address in readerwin.
Patch attached

Created attachment 2903 [details] 
Adds icons to context menu on address in readerwin
Adds icons to context menu on address in readerwin

*** Bug 66739 has been marked as a duplicate of this bug. ***

Why not adding those icons?

Why not adding those icons?
Without them this menu looks like no KDE menu.
OK, if you dont' want it, close it - I won't do it.

SVN commit 627763 by winterz:
merge SVN commit 627761 by winterz:
Fix bug "Add icons for context menu on address in readerwin"
Thanks for the patch Mikolaj. Hey, we only took 3+ years to commit it! :)
BUGS: 66740
 M +5 -5 kmreaderwin.cpp 
--- branches/KDE/3.5/kdepim/kmail/kmreaderwin.cpp #627762:627763
@@ -636,13 +636,13 @@
 mSelectEncodingAction->setItems( encodings );
 mSelectEncodingAction->setCurrentItem( 0 );
 
- mMailToComposeAction = new KAction( i18n("New Message To..."), 0, this,
- SLOT(slotMailtoCompose()), ac,
+ mMailToComposeAction = new KAction( i18n("New Message To..."), "mail_new",
+ 0, this, SLOT(slotMailtoCompose()), ac,
 "mailto_compose" );
- mMailToReplyAction = new KAction( i18n("Reply To..."), 0, this,
-				 SLOT(slotMailtoReply()), ac,
+ mMailToReplyAction = new KAction( i18n("Reply To..."), "mail_reply",
+ 0, this, SLOT(slotMailtoReply()), ac,
 				 "mailto_reply" );
- mMailToForwardAction = new KAction( i18n("Forward To..."),
+ mMailToForwardAction = new KAction( i18n("Forward To..."), "mail_forward",
 0, this, SLOT(slotMailtoForward()), ac,
 "mailto_forward" );
 mAddAddrBookAction = new KAction( i18n("Add to Address Book"),

Dnia niedziela 28 styczeń 2007, Allen Winter napisał:
> Resolution| |FIXED
>
> --- branches/KDE/3.5/kdepim/kmail/kmreaderwin.cpp #627762:627763
Thanks. I hope it will be ported to trunk :)