D6226: KKeySequenceWidget: make it possible to record Ctrl+Num+1 as a shortcut.

David Edmundson noreply at phabricator.kde.org
Sun Jun 18 16:04:54 UTC 2017


davidedmundson added inline comments.

INLINE COMMENTS

> kkeysequencewidget.cpp:778
>          if (keyQt) {
>              if ((keyQt == Qt::Key_Backtab) && (d->modifierKeys & Qt::SHIFT)) {
>                  keyQt = Qt::Key_Tab | d->modifierKeys;

Should this section be using ShiftModifier? Qt::SHIFT has been masked out

REPOSITORY
  R263 KXmlGui

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

To: dfaure, aacid
Cc: davidedmundson, apol, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170618/b11da016/attachment.html>


More information about the Kde-frameworks-devel mailing list