diff options
| author | Alex Miranda <[email protected]> | 2019-10-21 21:39:30 +0200 |
|---|---|---|
| committer | Elvis Angelaccio <[email protected]> | 2019-10-21 21:39:30 +0200 |
| commit | d8c11a67bd98820fd2c9a4b616460857af70344b (patch) | |
| tree | c2c6125d80f278f15e0ccb56a3ce9d80bdfd9e07 /src/dolphintabwidget.cpp | |
| parent | 0a93454c1debad767ea1fb97f23a9335cfc55c7f (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.cpp | 3 |
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, |
