Menu locator filter: Fix that e.g. recent files were no longer available

Amends aed2c2c716
After checking which actions are enabled we may not trigger the menus'
aboutToShow anymore, since this can completely rebuild the menu
including recreating the actions. We also don't have to, since this
moved to before checking which actions are enabled.

Change-Id: I1108edce0818f71080b2926a1121c45de38672e1
Reviewed-by: David Schulz <david.schulz@qt.io>
This commit is contained in:
Eike Ziller
2018-10-04 11:02:26 +02:00
parent 179d8dce73
commit 3a09687921

View File

@@ -190,8 +190,6 @@ void Core::Internal::MenuBarFilter::prepareSearch(const QString &entry)
const QStringList entryPath = normalized.split(separators.at(0), QString::SkipEmptyParts);
m_entries.clear();
QVector<const QMenu *> processedMenus;
for (QAction* action : menuBarActions()) {
requestMenuUpdate(action);
for (QAction* action : menuBarActions())
m_entries << matchesForAction(action, entryPath, QStringList(), processedMenus);
}
}