mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-27 20:58:47 +00:00
Code Issues Projects Releases Packages Wiki Activity

preferences: connect() dialog page only once

Browse source 
Weirdly, the settingsChanged() signal of the dialog-pages was
connected() to the settingsChanged() signal of the dialog
every time the settings were accepted. Do it only once
in the constructor.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2020-11-24 14:10:53 +01:00 committed by Dirk Hohndel
parent 0fab09a990
commit 712e4680ca
2 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
desktop-widgets/preferences/preferencesdialog.cpp
View file

@ -84,6 +84,7 @@ PreferencesDialog::PreferencesDialog()
pagesList->addItem(item);
pagesStack->addWidget(page);
page->refreshSettings();
connect(page, &AbstractPreferencesWidget::settingsChanged, &diveListNotifier, &DiveListNotifier::settingsChanged);
}
connect(pagesList, &QListWidget::currentRowChanged,
@ -116,10 +117,8 @@ void PreferencesDialog::refreshPages()
void PreferencesDialog::applyRequested(bool closeIt)
{
Q_FOREACH(AbstractPreferencesWidget *page, pages) {
connect(page, &AbstractPreferencesWidget::settingsChanged, &diveListNotifier, &DiveListNotifier::settingsChanged, Qt::UniqueConnection);
for (AbstractPreferencesWidget *page: pages)
page->syncSettings();
}
emit diveListNotifier.settingsChanged();
if (closeIt)
accept();

3
desktop-widgets/preferences/preferencesdialog.h
View file

@ -3,7 +3,6 @@
#define PREFERENCES_WIDGET_H
#include <QDialog>
#include "core/pref.h"
class AbstractPreferencesWidget;
class QListWidget;
@ -14,7 +13,7 @@ class QAbstractButton;
class PreferencesDialog : public QDialog {
Q_OBJECT
public:
static PreferencesDialog* instance();
static PreferencesDialog *instance();
~PreferencesDialog();
void refreshPages();
void defaultsRequested();