Bug 227540

Summary: JJ: checkbox instead of radiobox in opacity settings
Product: [Plasma] kwin Reporter: Felix Möller <felix>
Component: generalAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist CC: caiqian, gkaukola
Priority: NOR Keywords: junior-jobs
Version: unspecified   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Unspecified   
Latest Commit: Version Fixed In:
Attachments: Fixes fish user/group delimiter problem
Group the opacity actions.

Description Felix Möller 2010-02-18 17:42:18 UTC
Version:            (using KDE 4.4.0)
Installed from:    Fedora RPMs

See attached screenshot.

Just one entry can be selected so this has to be a radio box.
Comment 1 CAI Qian 2010-02-28 13:52:31 UTC
Where is the attachment?
Comment 2 Christoph Feck 2010-02-28 13:57:32 UTC
We don't need a screen shot, as that bug is easily reproducible.
Comment 3 Glen Kaukola 2010-03-16 04:36:44 UTC
Created attachment 41677 [details]
Fixes fish user/group delimiter problem

Fixes fish user/group delimiter problem.
Comment 4 Christoph Feck 2010-03-16 04:51:48 UTC
The patch is for bug 227678 ...
Comment 5 Glen Kaukola 2010-03-16 05:14:13 UTC
Yeah, sorry about that.  I blew it.
Comment 6 Glen Kaukola 2010-03-16 06:55:39 UTC
Actually a screenshot would have helped me figure out what this was about.  Anyhow, I'm attaching a patch for this one.  Any criticism will be appreciated.

Seems to me "To Desktop" could use the same treatment.  And perhaps "Move window to group" as well.  Although I have no idea how to create window groups at this point so I can't say for sure.
Comment 7 Glen Kaukola 2010-03-16 07:00:17 UTC
Created attachment 41679 [details]
Group the opacity actions.
Comment 8 Martin Flöser 2010-03-20 14:57:35 UTC
SVN commit 1105530 by graesslin:

Use radiobuttons instead of checkboxes in opacity user actions menu.
Thanks to Glen Kaukola for the patch.
FEATURE: 227540

 M  +2 -0      useractions.cpp  
 M  +2 -0      workspace.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1105530