D17442: KTextEditor: Tweak keyboard shortcuts to prepare for F6/Shift+F6 in Kate

[gregormi]

gregormi added a comment.


  With respect to bringing this https://phabricator.kde.org/T10279 forward, how should I proceed here? Can this go in as first step (after I removed F10 and F11)? (see also https://phabricator.kde.org/D17443)

INLINE COMMENTS

> kateview.cpp:700
>      ac->addAction(QStringLiteral("view_dynamic_word_wrap"), a);
> -    ac->setDefaultShortcut(a, QKeySequence(Qt::Key_F10));
> +    ac->setDefaultShortcuts(a, { QKeySequence(Qt::Key_F10), QKeySequence(Qt::CTRL + Qt::Key_E, Qt::CTRL + Qt::Key_W) });
>      a->setWhatsThis(i18n("If this option is checked, the text lines will be wrapped at the view border on the screen."));

todo: Remove F10

> kateview.cpp:727
>      ac->addAction(QStringLiteral("view_line_numbers"), a);
> -    ac->setDefaultShortcut(a, QKeySequence(Qt::Key_F11));
> +    ac->setDefaultShortcuts(a, { QKeySequence(Qt::Key_F11), QKeySequence(Qt::CTRL + Qt::Key_E, Qt::CTRL + Qt::Key_L) });
>      a->setWhatsThis(i18n("Show/hide the line numbers on the left hand side of the view."));

todo: remove F11

REPOSITORY
  R39 KTextEditor

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

To: gregormi, #kate
Cc: cullmann, dhaumann, ngraham, kwrite-devel, kde-frameworks-devel, hase, michaelh, bruns, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190112/dc6c1c5b/attachment-0001.html>