From c57c5384fcab92a1ba0d8fd87321ce7cb148ca02 Mon Sep 17 00:00:00 2001 From: Forest ix Date: Sun, 5 Feb 2023 18:38:55 -0800 Subject: [PATCH] Reveal the Backspace shortcut in the UI Since the Back action already has two standard shortcuts, a third one appended to the list will be invisible in the UI, which shows only the first two. This leaves users unable to reassign it. We therefore make it the first shortcut in the list, so that the hidden third entry will be the dedicated Back button, which users are far less likely to want reassigned. BUG: 371130 --- src/dolphinmainwindow.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp index 1804f34591..4740b66e9d 100644 --- a/src/dolphinmainwindow.cpp +++ b/src/dolphinmainwindow.cpp @@ -1786,7 +1786,8 @@ void DolphinMainWindow::setupActions() actionCollection()->addAction(m_backAction->objectName(), m_backAction); auto backShortcuts = m_backAction->shortcuts(); - backShortcuts.append(QKeySequence(Qt::Key_Backspace)); + // Prepend this shortcut, to avoid being hidden by the two-slot UI (#371130) + backShortcuts.prepend(QKeySequence(Qt::Key_Backspace)); actionCollection()->setDefaultShortcuts(m_backAction, backShortcuts); DolphinRecentTabsMenu *recentTabsMenu = new DolphinRecentTabsMenu(this);