diff options
| author | Edward Welbourne <edward.welbourne@qt.io> | 2023-06-14 16:18:32 +0200 |
|---|---|---|
| committer | Edward Welbourne <edward.welbourne@qt.io> | 2023-06-16 21:37:11 +0200 |
| commit | 3ccf2f8308ba33cab575c22ad2e246b987a3dc0c (patch) | |
| tree | 361a85e948c6bea55dd816b86f12c903775b7902 /examples/widgets | |
| parent | 59d28697655cbd41baa40804cdc43c96983261d5 (diff) | |
Examples: "About &Qt" should show info about Qt, not quit
A few examples seem to have copied and pasted a help action "About
&Qt" that triggered QCoreApplication::quit. This does not look like
best practice. Use QApplication::aboutQt instead.
Pick-to: 6.6 6.5
Task-number: QTBUG-111228
Change-Id: I1887a3c999d752a24c7c4d3cabc4a5d63b29b966
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Diffstat (limited to 'examples/widgets')
| -rw-r--r-- | examples/widgets/tools/settingseditor/mainwindow.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/examples/widgets/tools/settingseditor/mainwindow.cpp b/examples/widgets/tools/settingseditor/mainwindow.cpp index 68178a3747..be9f19e8cc 100644 --- a/examples/widgets/tools/settingseditor/mainwindow.cpp +++ b/examples/widgets/tools/settingseditor/mainwindow.cpp @@ -151,7 +151,7 @@ void MainWindow::createActions() QMenu *helpMenu = menuBar()->addMenu(tr("&Help")); helpMenu->addAction(tr("&About"), this, &MainWindow::about); - helpMenu->addAction(tr("About &Qt"), qApp, &QCoreApplication::quit); + helpMenu->addAction(tr("About &Qt"), qApp, &QApplication::aboutQt); } void MainWindow::setSettingsObject(const SettingsPtr &settings) |