┌   ┐
54
└   ┘

summaryrefslogtreecommitdiff
path: root/src/dolphintabwidget.cpp
diff options
context:
space:
mode:
authorAlex Miranda <[email protected]>2019-10-21 21:39:30 +0200
committerElvis Angelaccio <[email protected]>2019-10-21 21:39:30 +0200
commitd8c11a67bd98820fd2c9a4b616460857af70344b (patch)
treec2c6125d80f278f15e0ccb56a3ce9d80bdfd9e07 /src/dolphintabwidget.cpp
parent0a93454c1debad767ea1fb97f23a9335cfc55c7f (diff)
Disable keyboard accelerators for the tabs widget
Summary: Now that there are actions for switching to a specific tab with default Alt + <number> shortcuts, automatically assigned keyboard accelerators can result in ambiguous shortcuts when using specific tab names containing numbers. Not adding any accelerators for the tabs widget ensures the default shortcuts work reliably regardless of tab names. Test Plan: - Verify there are no ambiguous shortcuts when browsing folders with names containing only numbers - Verify other keyboard accelerators still work (menu bar, context menu and others) Reviewers: #vdg, #dolphin, ngraham Reviewed By: #vdg, ngraham Subscribers: ngraham, kfm-devel Tags: #dolphin Differential Revision: https://phabricator.kde.org/D24794
Diffstat (limited to 'src/dolphintabwidget.cpp')
-rw-r--r--src/dolphintabwidget.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/dolphintabwidget.cpp b/src/dolphintabwidget.cpp
index 0408d7ed4..9ecc14427 100644
--- a/src/dolphintabwidget.cpp
+++ b/src/dolphintabwidget.cpp
@@ -28,6 +28,7 @@
#include <KRun>
#include <KShell>
#include <kio/global.h>
+#include <KAcceleratorManager>
#include <QApplication>
#include <QDropEvent>
@@ -37,6 +38,8 @@ DolphinTabWidget::DolphinTabWidget(QWidget* parent) :
m_placesSelectorVisible(true),
m_lastViewedTab(0)
{
+ KAcceleratorManager::setNoAccel(this);
+
connect(this, &DolphinTabWidget::tabCloseRequested,
this, QOverload<int>::of(&DolphinTabWidget::closeTab));
connect(this, &DolphinTabWidget::currentChanged,