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
This commit is contained in:
Forest ix 2023-02-05 18:38:55 -08:00 committed by Méven Car
parent 34e0a5488e
commit c57c5384fc

View file

@ -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);