[Kde-pim] branches/work/kdepim-3.5.5+/kmail

[Adriaan de Groot]

SVN commit 633861 by adridg:

If there is no template to use for an action, disable the menu item itself instead of inserting a lame 1-disabled-item submenu. (Well, insert the lame submenu anyway, but disable the menu item too.) Still waiting on a real usability answer on this issue, but the votes in #kontact go for this.

CCMAIL: kde-pim at kde.org


 M  +3 -0      branches/work/kdepim-3.5.5+/kmail/kmmainwidget.cpp  


--- branches/work/kdepim-3.5.5+/kmail/kmmainwidget.cpp #633860:633861
@@ -2552,14 +2552,17 @@
   if ( !replyc ) {
       mCustomReplyActionMenu->popupMenu()->insertItem( i18n( "(no templates)" ), 0 );
       mCustomReplyActionMenu->popupMenu()->setItemEnabled( 0, false );
+      mCustomReplyActionMenu->setEnabled(false);
   }
   if ( !replyallc ) {
       mCustomReplyAllActionMenu->popupMenu()->insertItem( i18n( "(no templates)" ), 0 );
       mCustomReplyAllActionMenu->popupMenu()->setItemEnabled( 0, false );
+      mCustomReplyAllActionMenu->setEnabled(false);
   }
   if ( !forwardc ) {
       mCustomForwardActionMenu->popupMenu()->insertItem( i18n( "(no templates)" ), 0 );
       mCustomForwardActionMenu->popupMenu()->setItemEnabled( 0, false );
+      mCustomForwardActionMenu->setEnabled(false);
   }
 
 }
_______________________________________________
kde-pim mailing list
kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/