From d65e6e9709a194f7f0430dc64ec199a9b337cd76 Mon Sep 17 00:00:00 2001 From: Kurt Hindenburg Date: Sat, 28 Mar 2020 12:23:22 -0400 Subject: [PATCH] Remove shortcut for detach tab Currently the shortcut is Ctrl+Shift+L which is close to Ctrl+Shift+K. Ideally we want detach tab to not be a shortcut that can be accidently pressed. See https://invent.kde.org/kde/konsole/-/issues/6 --- src/ViewManager.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/ViewManager.cpp b/src/ViewManager.cpp index 871504040..3d211bb1a 100644 --- a/src/ViewManager.cpp +++ b/src/ViewManager.cpp @@ -158,9 +158,6 @@ void ViewManager::setupActions() action->setText(i18nc("@action:inmenu", "Detach Current &Tab")); connect(action, &QAction::triggered, this, &ViewManager::detachActiveTab); _multiTabOnlyActions << action; - // Ctrl+Shift+D is not used as a shortcut by default because it is too close - // to Ctrl+D - which will terminate the session in many cases - collection->setDefaultShortcut(action, Konsole::ACCEL + Qt::SHIFT + Qt::Key_L); // keyboard shortcut only actions action = new QAction(i18nc("@action Shortcut entry", "Next Tab"), this);