Editor: avoid accessing the widget from the FontSettingsPage

Change-Id: I51dbf4d1c7384f460b84da9602c10667addabebb
Reviewed-by: hjk <hjk@qt.io>

1 files changed, 8 insertions, 6 deletions

diff --git a/src/plugins/texteditor/fontsettingspage.cpp b/src/plugins/texteditor/fontsettingspage.cpp
index b57e23f35b..4488986405 100644
--- a/src/plugins/texteditor/fontsettingspage.cpp
+++ b/src/plugins/texteditor/fontsettingspage.cpp
@@ -202,6 +202,8 @@ public:
                 this, &FontSettingsPageWidget::importScheme);
         connect(exportButton, &QPushButton::clicked,
                 this, &FontSettingsPageWidget::exportScheme);
+        connect(TextEditorSettings::instance(), &TextEditorSettings::fontSettingsChanged,
+                this, &FontSettingsPageWidget::updateFontZoom);
 
         updatePointSizes();
         refreshColorSchemeList();
@@ -226,6 +228,7 @@ public:
 
     void maybeSaveColorScheme();
     void updatePointSizes();
+    void updateFontZoom(const FontSettings &fontSettings);
     QList<int> pointSizesForSelectedFont() const;
     void refreshColorSchemeList();
 
@@ -449,6 +452,11 @@ void FontSettingsPageWidget::updatePointSizes()
         m_sizeComboBox->setCurrentIndex(idx);
 }
 
+void FontSettingsPageWidget::updateFontZoom(const FontSettings &fontSettings)
+{
+    m_zoomSpinBox->setValue(fontSettings.fontZoom());
+}
+
 QList<int> FontSettingsPageWidget::pointSizesForSelectedFont() const
 {
     QFontDatabase db;
@@ -774,10 +782,4 @@ FontSettingsPage::FontSettingsPage(FontSettings *fontSettings, const FormatDescr
     setWidgetCreator([this, fontSettings, fd] { return new FontSettingsPageWidget(this, fd, fontSettings); });
 }
 
-void FontSettingsPage::setFontZoom(int zoom)
-{
-    if (m_widget)
-        static_cast<FontSettingsPageWidget *>(m_widget.data())->m_zoomSpinBox->setValue(zoom);
-}
-
 } // TextEditor