kxmlgui shortcuts regression

David Faure faure at kde.org
Mon Sep 22 21:34:28 BST 2008

On Saturday 20 September 2008, Nick Shaforostoff wrote:
> >Never call setShortcut over an QAction* if the action is a KAction.
> Why not subscribe to QAction::changed() signal and check if shortcut
> was changed?

Hmm. This would 
1) still not support KShortcut, right? QAction::setShortcut() only takes a QKeySequence.
2) also be called when KAction calls QAction::setShortcut[s], so we would need
some boolean to avoid recursion.

But if we want to support this, then indeed it's a way to do it.

David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).

More information about the kde-core-devel mailing list