Review Request: correct confusing action names in kanagrams shortcuts dialog

Burkhard Lück lueck at hube-lueck.de
Fri Jun 3 09:00:58 CEST 2011


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

Review request for KDE Edu and Jeremy Paul Whiting.


Summary
-------

Kanagram has KStandardAction::prior (Qt::Key_PageUp) and next (Qt::Key_PageDown) for the actions to switch to the previous/next Vocabulary. But due to using KStandardAction::prior() and KStandardAction::next() these actions are labelled "Previous Page" and "Next Page" in the shortcuts dialog.
Patch renames them to "Previous Vocabulary" and "Next Vocabulary" to make it clear, which action are handled with these shortcuts.


Diffs
-----

  src/kanagram.cpp 97e1e90 

Diff: http://git.reviewboard.kde.org/r/101496/diff


Testing
-------


Thanks,

Burkhard

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-edu/attachments/20110603/b7321561/attachment.htm 


More information about the kde-edu mailing list