Add Copy and Select-All Shortcuts to Choqok
Needs ReviewPublic

Authored by braxtonsalyer on Mar 17 2018, 9:24 PM.

Details

Reviewers
scarpino
Group Reviewers
Choqok
Summary

BUG: 251164

Adds Copy and Select-All Shortcuts to Choqok using KStandardShortcuts from KConfigGui

Diff Detail

Repository
R186 Choqok
Lint
Lint Skipped
Unit
Unit Tests Skipped
braxtonsalyer requested review of this revision.Mar 17 2018, 9:24 PM
braxtonsalyer created this revision.
scarpino requested changes to this revision.Mar 19 2018, 8:32 AM
scarpino added a subscriber: scarpino.
scarpino added inline comments.
libchoqok/ui/textbrowser.cpp
122

Please use QAction::setShortcuts() instead.

133

Please use QAction::setShortcuts() instead.

This revision now requires changes to proceed.Mar 19 2018, 8:32 AM
scarpino retitled this revision from Fix Bug 251164 - Add Copy and Select-All Shortcuts to Choqok to Add Copy and Select-All Shortcuts to Choqok.Mar 19 2018, 8:42 AM
scarpino edited the summary of this revision. (Show Details)

Ammended to use QAction::setShortcuts() instead of QAction::setShortcut()