Members of the KDE Community are recommended to subscribe to the kde-community mailing list at https://mail.kde.org/mailman/listinfo/kde-community to allow them to participate in important discussions and receive other important announcements

Commit ab1f4721 authored by Huon Imberger's avatar Huon Imberger Committed by Henrik Fehlauer

Save config when opening the config dialog

Summary:
Some settings can be changed outside of the config dialog. We don't
want these changes to be lost when changing the config, which
triggers reloading of the config.

BUG: 390331

Test Plan:
Settings changed outside the config dialog, e.g. thumbnail zoom level in Browse view,
should not reset if you change a setting in the config dialog.

Reviewers: #gwenview, rkflx

Reviewed By: #gwenview, rkflx

Differential Revision: https://phabricator.kde.org/D10626
parent 7dbda5ae
......@@ -1438,6 +1438,9 @@ bool MainWindow::queryClose()
void MainWindow::showConfigDialog()
{
// Save first so changes like thumbnail zoom level are not lost when reloading config
saveConfig();
DialogGuard<ConfigDialog> dialog(this);
connect(dialog.data(), SIGNAL(settingsChanged(QString)), SLOT(loadConfig()));
dialog->exec();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment