[Kde-games-devel] Review Request 127856: Use KActionCollection::setDefaultShortcut for select_carddeck shortcut (F10)

Martin Walch walch.martin at web.de
Sat May 7 11:47:07 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127856/
-----------------------------------------------------------

Review request for KDE Games.


Repository: lskat


Description
-------

KXMLGuiFactory complains:

> Shortcut for action "select_carddeck" "Select &Card Deck..." set with
> QAction::setShortcut()! Use KActionCollection::setDefaultShortcut(s) instead.

Do so accordingly.

This also moves away from deprecated KShortcut.


Diffs
-----

  src/mainwindow.cpp b2c6d6f 

Diff: https://git.reviewboard.kde.org/r/127856/diff/


Testing
-------

Builds fine, does not show deprecation warning about KShortcut any more.

Shortcut F10 at runtime still working.


Thanks,

Martin Walch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20160507/f60407b4/attachment.html>


More information about the kde-games-devel mailing list