mirror of
https://github.com/keepassxreboot/keepassxc.git
synced 2026-03-11 08:54:48 +00:00
Fix delete plugin menu action placement
Make comment clearer.
This commit is contained in:
parent
6036e7fa7b
commit
27cd97a4a8
2 changed files with 3 additions and 2 deletions
|
|
@ -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) {
|
||||
|
|
|
|||
|
|
@ -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,
|
||||
|
|
|
|||
Loading…
Reference in a new issue