D10668: Fix PowerDevil shortcuts migration

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> aacid wrote in powerdevilapp.cpp:177
> Won't this return QList<QKeySequence>() since you've set it on the line just before?

You'd think so, but no.

setShortcut(QAction, sequence, KGlobalAccel::NoAutoloading)   
would do exactly that

setShortcut(QAction, sequence, KGlobalAccel::Autoloading)   
(which this defaults to)
will make KGlobalAccel load whatever it has stored already in its config file.

REPOSITORY
  R122 Powerdevil

REVISION DETAIL
  https://phabricator.kde.org/D10668

To: davidedmundson, #plasma
Cc: aacid, cfeck, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180219/b6bac48c/attachment.html>