Review Request: Fix KShortcut to really allow the usage of multiple shortcuts

Michael Jansen kde at
Tue May 1 20:17:14 BST 2012

On Tuesday, May 01, 2012 06:05:01 PM Mark Gaiser wrote:

This is an automatically generated e-mail. To reply, visit: 

On April 30th, 2012, 10:19 p.m., David Faure wrote: 
I am in favour of the idea, since I was hit by this limitation in the past, 
However I think the API changes should be more conservative, there's no need 
to deprecate so many of the existing methods, just because it is *possible* to 
define more than 2 shortcuts.

Also, Qt's QAction already has void QAction::setShortcuts(const 
QList<QKeySequence> &shortcuts)... remind me why we are not using that? 
Because it doesn't know "default shortcut" vs "current shortcut"? 
I would love to remind you, but i simply don't know. I took it for granted 
since the current documentation talks about it. The doc above 
(line 316)

Most likely because it says since Qt 4.2 in the docs and KAction was done 
before that?

Before my time too.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list