https://bugs.kde.org/show_bug.cgi?id=447452

            Bug ID: 447452
           Summary: [plasma-interactiveconsole] Ctrl+E to Execute is an
                    ambiguous shortcut
           Product: Plasma SDK
           Version: 5.23.4
          Platform: Other
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: General
          Assignee: plasma-b...@kde.org
          Reporter: zrenf...@gmail.com
                CC: antonis.tsiapalio...@kde.org
  Target Milestone: ---

This is technically a bug for plasma-workspace, since that's where
plasma-interactiveconsole is.

*
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1293#note_363801
*
https://invent.kde.org/plasma/plasma-workspace/-/tree/master/interactiveconsole
*
https://invent.kde.org/plasma/plasma-workspace/-/blob/master/interactiveconsole/interactiveconsole.cpp#L188

Looks like it's bound to Ctrl+E, but pressing that shows that the shortcut is
Ambiguous. Looks like you need to not be focusing the code TextArea (select
text in the output TextArea) for Ctrl+E to work, which defeats the purpose of
the shortcut. Also, there is no settings menu (probably meant for Kate).

Ctrl+B seems to be bound to... Toggle Breakpoint or something? It turns the bg
blue on the line with the cursor.

We should probably keep `Alt+E` generated by the i18n text. So we should be
using something like:

    auto shortcuts = QList<QKeySequence>() << m_executeAction->shortcut() <<
(Qt::CTRL | Qt::Key_E) << (Qt::CTRL | Qt::Key_B);
    m_executeAction->setShortcuts(shortcuts);

We'll also need to figure out how to disable the default KTextEditor Ctrl+B /
Ctrl+E actions so they're not ambiguous.

* https://doc.qt.io/qt-5/qaction.html#shortcuts

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to