[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Support in KToggleAction for toggling the text itself
From:       Martijn Klingens <klingens () kde ! org>
Date:       2004-02-07 11:07:30
Message-ID: 200402071207.30568.klingens () kde ! org
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]