From 28b06e89d54b95d83abcbf6357f3e61a9c5131b6 Mon Sep 17 00:00:00 2001 From: hjk Date: Tue, 9 Jan 2024 13:18:00 +0100 Subject: [PATCH] Core: Use ActionBuilder in FolderNavigationWidgetFactory Change-Id: I1f113724c2c9aba6657bddcc9ecfa9c0f151a7e8 Reviewed-by: Eike Ziller --- .../coreplugin/foldernavigationwidget.cpp | 41 ++++++++++--------- 1 file changed, 22 insertions(+), 19 deletions(-) diff --git a/src/plugins/coreplugin/foldernavigationwidget.cpp b/src/plugins/coreplugin/foldernavigationwidget.cpp index b58b6c4e466..30505bc0cf1 100644 --- a/src/plugins/coreplugin/foldernavigationwidget.cpp +++ b/src/plugins/coreplugin/foldernavigationwidget.cpp @@ -926,28 +926,31 @@ void FolderNavigationWidgetFactory::removeRootPath(Utils::Id id) void FolderNavigationWidgetFactory::registerActions() { - Core::Context context(C_FOLDERNAVIGATIONWIDGET); + const Context context(C_FOLDERNAVIGATIONWIDGET); - auto add = new QAction(Tr::tr("Add New..."), this); - Core::ActionManager::registerAction(add, ADDNEWFILE, context); - connect(add, &QAction::triggered, Core::ICore::instance(), [] { - if (auto navWidget = currentFolderNavigationWidget()) - navWidget->addNewItem(); - }); + ActionBuilder(this, ADDNEWFILE) + .setText(Tr::tr("Add New...")) + .setContext(context) + .addOnTriggered([] { + if (auto navWidget = currentFolderNavigationWidget()) + navWidget->addNewItem(); + }); - auto rename = new QAction(Tr::tr("Rename..."), this); - Core::ActionManager::registerAction(rename, RENAMEFILE, context); - connect(rename, &QAction::triggered, Core::ICore::instance(), [] { - if (auto navWidget = currentFolderNavigationWidget()) - navWidget->editCurrentItem(); - }); + ActionBuilder(this, RENAMEFILE) + .setText(Tr::tr("Rename...")) + .setContext(context) + .addOnTriggered([] { + if (auto navWidget = currentFolderNavigationWidget()) + navWidget->editCurrentItem(); + }); - auto remove = new QAction(Tr::tr("Remove..."), this); - Core::ActionManager::registerAction(remove, REMOVEFILE, context); - connect(remove, &QAction::triggered, Core::ICore::instance(), [] { - if (auto navWidget = currentFolderNavigationWidget()) - navWidget->removeCurrentItem(); - }); + ActionBuilder(this, REMOVEFILE) + .setText(Tr::tr("Remove...")) + .setContext(context) + .addOnTriggered([] { + if (auto navWidget = currentFolderNavigationWidget()) + navWidget->removeCurrentItem(); + }); } int DelayedFileCrumbLabel::immediateHeightForWidth(int w) const