From a4b3085acb90652cc1fa867f8e0832e887f2bbfa Mon Sep 17 00:00:00 2001 From: Eike Ziller Date: Tue, 25 Jan 2022 14:17:55 +0100 Subject: [PATCH] Fix issues with "Select All" (search results) - the shortcut Alt+Return conflicts with "Trigger Refactoring Action" - display the shortcut in find toolbar button's tooltip - no need to repeat the default shortcut (it's the same Command) Change-Id: Ifd1e56ba091806901241bdca66c42a332574bc0c Reviewed-by: David Schulz --- src/plugins/coreplugin/find/findtoolbar.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/plugins/coreplugin/find/findtoolbar.cpp b/src/plugins/coreplugin/find/findtoolbar.cpp index 4a374fa742e..f7345f28c9c 100644 --- a/src/plugins/coreplugin/find/findtoolbar.cpp +++ b/src/plugins/coreplugin/find/findtoolbar.cpp @@ -224,12 +224,12 @@ FindToolBar::FindToolBar(CurrentDocumentFind *currentDocumentFind) m_selectAllAction = new QAction(tr("Select All"), this); cmd = ActionManager::registerAction(m_selectAllAction, Constants::FIND_SELECT_ALL); - cmd->setDefaultKeySequence(QKeySequence(tr("Alt+Return"))); + cmd->setDefaultKeySequence(QKeySequence(tr("Ctrl+Alt+Return"))); mfind->addAction(cmd, Constants::G_FIND_ACTIONS); connect(m_selectAllAction, &QAction::triggered, this, &FindToolBar::selectAll); m_localSelectAllAction = new QAction(m_selectAllAction->text(), this); cmd = ActionManager::registerAction(m_localSelectAllAction, Constants::FIND_SELECT_ALL, findcontext); - cmd->setDefaultKeySequence(QKeySequence(tr("Alt+Return"))); + cmd->augmentActionWithShortcutToolTip(m_localSelectAllAction); connect(m_localSelectAllAction, &QAction::triggered, this, &FindToolBar::selectAll); m_ui.selectAllButton->setDefaultAction(m_localSelectAllAction);