Bug 229135 - checkable group boxes should get an accelerator
Summary: checkable group boxes should get an accelerator
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: shortcuts (show other bugs)
Version: 4.4
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-02 15:34 UTC by Jonathan Marten
Modified: 2010-03-03 14:11 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Marten 2010-03-02 15:34:13 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

The automatic assignment of accelerators to widgets (KAcceleratorManager and KAccelManagerAlgorithm) does not assign a shortcut to a checkable group box.  For a standard group box this is pointless, but a checkable group box should be considered equivalent to a QCheckBox and be assigned an accelerator for user interaction.
Comment 1 Jonathan Marten 2010-03-02 15:51:31 UTC
Patch to fix and review at http://reviewboard.kde.org/r/3084/
Comment 2 Jonathan Marten 2010-03-03 14:11:34 UTC
SVN commit 1098311 by marten:

Automatically assign an accelerator to a checkable group box (because
it is useful for user interaction).  But not to a normal group box
(because it's pointless).

Do this at a lower priority than other widget types, so that existing
assignments should not change too much.

Review at http://reviewboard.kde.org/r/3084/

BUG:229135


 M  +15 -5     kacceleratormanager.cpp  
 M  +3 -1      kacceleratormanager_private.h  


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