Fix currentPage QPROPERTY missing NOTIFY in settingsdialog class

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
nextcloud · Jan 2, 2023 · 403f43b · 403f43b
1 parent 0495c65
commit 403f43b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/gui/settingsdialog.cpp b/src/gui/settingsdialog.cpp
@@ -132,6 +132,8 @@ SettingsDialog::SettingsDialog(ownCloudGui *gui, QWidget *parent)
     auto *networkSettings = new NetworkSettings;
     _ui->stack->addWidget(networkSettings);
 
+    connect(_ui->stack, &QStackedWidget::currentChanged, this, &SettingsDialog::currentPageChanged);
+
     _actionGroupWidgets.insert(generalAction, generalSettings);
     _actionGroupWidgets.insert(networkAction, networkSettings);
 

diff --git a/src/gui/settingsdialog.h b/src/gui/settingsdialog.h
@@ -45,7 +45,7 @@ class ownCloudGui;
 class SettingsDialog : public QDialog
 {
     Q_OBJECT
-    Q_PROPERTY(QWidget* currentPage READ currentPage)
+    Q_PROPERTY(QWidget* currentPage READ currentPage NOTIFY currentPageChanged)
 
 public:
     explicit SettingsDialog(ownCloudGui *gui, QWidget *parent = nullptr);
@@ -63,6 +63,7 @@ public slots:
 signals:
     void styleChanged();
     void onActivate();
+    void currentPageChanged();
 
 protected:
     void reject() override;