feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

ExtensionManager: Use QPushButton::clicked, not pressed

Browse Source 
"Clicked" is "press and release", and we use that when reacting to
buttons. Opening the window on "pressed" leads to the "released" event
not being handled by the QPushButton, which confuses Qt on macOS and
leads to an ugly gray background staying around after using it.

Change-Id: Ieff15d222cdaeffd498594ca300e960421493940
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
This commit is contained in:
Eike Ziller
2024-10-07 10:32:50 +02:00
parent b788cf8ee5
commit b2114507da
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/plugins/extensionmanager/extensionsbrowser.cpp
View File

@@ -590,7 +590,7 @@ ExtensionsBrowser::ExtensionsBrowser(ExtensionsModel *model, QWidget *parent)
d->sortFilterProxyModel, &SortFilterProxyModel::setSortOption);
connect(d->filterChooser, &OptionChooser::currentIndexChanged,
d->sortFilterProxyModel, &SortFilterProxyModel::setFilterOption);
connect(settingsToolButton, &QAbstractButton::pressed, this, []() {
connect(settingsToolButton, &QAbstractButton::clicked, this, []() {
ICore::showOptionsDialog(Constants::EXTENSIONMANAGER_SETTINGSPAGE_ID);
});
connect(&settings(), &AspectContainer::changed, this, [this]() {