Whence the accelerators?
kfunk at kde.org
Fri May 5 10:31:44 BST 2017
On Friday, 5 May 2017 10:55:27 CEST Alexander Zhigalin wrote:
> 05.05.2017, 10:17, "Kevin Funk" <kfunk at kde.org>:
> On Thursday, 4 May 2017 21:40:00 CEST Matthew Woehlke wrote:
> In KDevelop 4.x, I got used to being able to right-click a project and
> hit 'b' to quickly build that project.
> In KDevelop 5.1.0, this doesn't work; in fact, the project context menu
> doesn't appear to have *any* accelerators. Did something happen to them,
> or have I turned them off somehow?
> This was a KMenu (from kdelibs) feature  -- which is deprecated nowadays.
> During the kdelibs (Qt4) -> KF5 (Qt5) transition we ported all uses of
> KMenu to QMenu, which doesn't have this feature.
>  See:
> You'd need to port KMenu's accelerator feature to QMenu to fix.
> But we can use the Alt key, right?
> Now in the project context menu (see screenshot) there are alt-bindings only
> for "copy" and "paste" entries...
And actually just opening that menu and pressing 'c' or 'p' has the desired
effect. So it's indeed just related to those other specific actions in that
menu. The feature in general works.
Seems like those actions just don't have proper accelerators in their titles.
I'd say: patches welcome!
ProjectManagerViewPlugin::contextMenuExtension( KDevelop::Context* context )
I think the key difference is that under kdelibs, KMenu generated accelerators
on the fly for all actions in the menu; while QMenu does not do that => thus
we have to add accelerators ourselves.
Again, patches or more debugging welcome, for those who'd like to have this
> Kevin Funk | kfunk at kde.org | http://kfunk.org
> Alexander Zhigalin - DevOps
Kevin Funk | kfunk at kde.org | http://kfunk.org
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 163 bytes
Desc: This is a digitally signed message part.
More information about the KDevelop