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

Reply via email to