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

List:       kde-bugs-dist
Subject:    [Bug 122016] keep window above/below all other windows cannot be
From:       Michael Brade <brade () kde ! org>
Date:       2006-02-15 12:46:43
Message-ID: 20060215124643.24062.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=122016         
brade kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From brade kde org  2006-02-15 13:46 -------
SVN commit 509655 by brade:

Oh no.. Tobias forgot to revert this in Osnabrück :-( So I'm doing it now
to fixe KNotes' "Keep Above Others"-toggle menu item, and probably others.
Without this it is not possible to uncheck the item anymore.

If you need to have at least one item checked in an exclusive group then
do it in the app's code by testing the action in the called slot.

BUG:122016


 M  +0 -4      kactionclasses.cpp  


--- branches/KDE/3.5/kdelibs/kdeui/kactionclasses.cpp #509654:509655
 @ -233,10 +233,6  @
 
 void KToggleAction::slotActivated()
 {
-  // don't toggle when already checked and part of exclusive group
-  if ( isChecked() && !exclusiveGroup().isEmpty() )
-    return;
-
   setChecked( !isChecked() );
   KAction::slotActivated();
   emit toggled( isChecked() );
[prev in list] [next in list] [prev in thread] [next in thread]