From kde-core-devel  Sat Feb 07 11:07:30 2004
From: Martijn Klingens <klingens () kde ! org>
Date: Sat, 07 Feb 2004 11:07:30 +0000
To: kde-core-devel
Subject: Re: Support in KToggleAction for toggling the text itself
Message-Id: <200402071207.30568.klingens () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=107619319910739

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