Summary: | Configure a keyboard shortcut by right clicking on a menu item does not work | ||
---|---|---|---|
Product: | [Unmaintained] kdelibs | Reporter: | Christos Gourdoupis <xrigou> |
Component: | kdeui | Assignee: | kdelibs bugs <kdelibs-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | cedric.laczny, cfeck, luke-jr+kdebugs, peter.penz19 |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Christos Gourdoupis
2009-12-13 11:29:56 UTC
SVN commit 1069933 by cfeck: Fix crash with context menus on KMenu CCBUG: 218485 CCBUG: 221123 This only fixes the crash, but the invoked action still does not work. There seems to be a Qt 4.6 regression. M +3 -0 kmenu.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1069933 SVN commit 1070420 by cfeck: Fix context menus on KMenu QMenu::activeAction() is already 0 when the action is emitted. BUG: 218485 BUG: 221123 M +6 -4 kmenu.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1070420 It is *almost* fixed. We can define shortcuts now. It doesn't work for View->Panels though. Which is how I run into this bug in the first place :) Peter, what could prevent configuring shortcuts for View->Panels actions from working? Other checkable actions work. SVN commit 1073053 by ppenz: Fixed issue that the keyboard shortcuts for the docks places, folders, information and terminal could not be changed. BUG: 218485 M +13 -8 dolphinmainwindow.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1073053 SVN commit 1073055 by ppenz: Backport of SVN commit SVN 1073053: Fixed issue that the keyboard shortcuts for the docks places, folders, information and terminal could not be changed. CCBUG: 218485 M +13 -8 dolphinmainwindow.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1073055 @Christoph: The QActions from QDock had been used directly, but they are not visible for the shortcuts dialog (an instance of KAction is required). I used now KActions that delegate the triggering to the corresponding QActions... Thanks Peter. I doubt it will ever be possible to support plain QAction in the shortcut dialog: // This code doesn't allow editing of QAction. It can not distinguish // between default and active shortcuts. This breaks many assumptions the // editor makes. I am still closing this as fixed; if there is really a need to support QAction in the dialog, it would have to be filed as a separate request. *** Bug 234589 has been marked as a duplicate of this bug. *** *** Bug 235549 has been marked as a duplicate of this bug. *** |