kxmlgui shortcuts regression

[David Faure]

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).