From 27cd97a4a862c960a3077821fe97f877c867c648 Mon Sep 17 00:00:00 2001 From: Agoston Szepessy Date: Mon, 29 Dec 2025 21:43:36 -0800 Subject: [PATCH] Fix delete plugin menu action placement Make comment clearer. --- src/gui/reports/ReportsWidgetBase.cpp | 2 +- src/gui/reports/ReportsWidgetBrowserStatistics.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gui/reports/ReportsWidgetBase.cpp b/src/gui/reports/ReportsWidgetBase.cpp index 3f3506b31..131114738 100644 --- a/src/gui/reports/ReportsWidgetBase.cpp +++ b/src/gui/reports/ReportsWidgetBase.cpp @@ -143,7 +143,7 @@ QMenu* ReportsWidgetBase::customMenuRequestedBase() // If we are including entries (checked is false) but a group is excluded, ask the user if they // would like to include the rest of the group as well (or keep it excluded). - // If they exclude it, we need to include the whole group, and then exclude + // If they choose "No", we need to include the whole group, and then exclude // the entries that aren't selected here. if (!checked) { for (const auto index : selected) { diff --git a/src/gui/reports/ReportsWidgetBrowserStatistics.cpp b/src/gui/reports/ReportsWidgetBrowserStatistics.cpp index 1e5d198cb..d2caeb066 100644 --- a/src/gui/reports/ReportsWidgetBrowserStatistics.cpp +++ b/src/gui/reports/ReportsWidgetBrowserStatistics.cpp @@ -245,7 +245,8 @@ void ReportsWidgetBrowserStatistics::customMenuRequested(QPoint pos) // Create the "delete plugin data" menu item const auto deletePluginData = new QAction(icons()->icon("entry-delete"), tr("Delete plugin data from Entry(s)…", "", selected.size()), this); - menu->addAction(deletePluginData); + menu->insertAction(menu->actions().at(3), + deletePluginData); // Index 3 is the one after "Delete Entry" so place "Delete plugin" before it connect(deletePluginData, &QAction::triggered, this,