On Friday 06 February 2004 18:05, David Faure wrote:
> The solution:
> --------------
> The attached patch adds support for (optionally) toggling the text instead
> of using a checkmark for toggleactions in menuitems, and uses this support
> for the show/hide menubar and toolbar actions.

Hmm, thinking about this... shouldn't this be a setCheckedGuiItem so you can 
pass both a text and an icon for the checked state?

Question is then what a null pixmap means, use the 'normal' pixmap, or no 
pixmap at all, since both have their uses.

-- 
Martijn