Review Request: Use a KDualAction to show/hide toolbar if there is only one

Aurélien Gâteau agateau at
Sun Oct 17 11:30:53 BST 2010

This is an automatically generated e-mail. To reply, visit:

Review request for kdelibs.


Now that the actions to show/hide menubar and statusbar are KDualAction, the action which show/hide the toolbar should be changed as well for consistency.

I only changed the case where there is only one toolbar because when there are multiple toolbars the action is turned into a submenu labeled "Toolbars Shown". It makes sense here to have toggle actions because the submenu only lists the name of the toolbars, without verbs.


  trunk/KDE/kdelibs/kdeui/xmlgui/ktoolbarhandler.cpp 1186567 



Tested with a single-toolbar application (KWrite) and a multi-toolbar one (Konqueror).



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list