From 67189f953d2af340a813acf30de062cabb1964a2 Mon Sep 17 00:00:00 2001
From: Mikhail Svechnikov <m.svechnikov@fz-juelich.de>
Date: Thu, 8 Sep 2022 12:49:18 +0200
Subject: [PATCH] JobRealTimeWidget: read only

---
 GUI/View/Fit/JobRealTimeWidget.cpp | 23 ++++-------------------
 GUI/View/Fit/JobRealTimeWidget.h   |  5 +----
 2 files changed, 5 insertions(+), 23 deletions(-)

diff --git a/GUI/View/Fit/JobRealTimeWidget.cpp b/GUI/View/Fit/JobRealTimeWidget.cpp
index 039aa1985d1..458c71697bc 100644
--- a/GUI/View/Fit/JobRealTimeWidget.cpp
+++ b/GUI/View/Fit/JobRealTimeWidget.cpp
@@ -27,7 +27,6 @@ const bool reuse_widget = true;
 JobRealTimeWidget::JobRealTimeWidget(JobModel* jobModel, QWidget* parent)
     : QWidget(parent)
     , m_stackedWidget(new ItemStackPresenter<ParameterTuningWidget>(reuse_widget))
-    , m_panelWidth(GUI::Constants::FIT_ACTIVITY_PANEL_WIDTH)
 {
     setWindowTitle(GUI::Constants::JobRealTimeWidgetName);
     setObjectName("JobRealTimeWidget");
@@ -41,13 +40,8 @@ JobRealTimeWidget::JobRealTimeWidget(JobModel* jobModel, QWidget* parent)
 
     m_stackedWidget->setModel(jobModel);
 
-    readSettings();
-    setFixedWidth(m_panelWidth);
-}
-
-JobRealTimeWidget::~JobRealTimeWidget()
-{
-    writeSettings();
+    setFixedWidth(GUI::Constants::FIT_ACTIVITY_PANEL_WIDTH);
+    applySettings();
 }
 
 void JobRealTimeWidget::resizeEvent(QResizeEvent*)
@@ -77,21 +71,12 @@ bool JobRealTimeWidget::isValidJobItem(JobItem* item)
     return item && (item->isCompleted() || item->isCanceled() || item->isFailed());
 }
 
-void JobRealTimeWidget::readSettings()
+void JobRealTimeWidget::applySettings()
 {
     QSettings settings;
     if (settings.childGroups().contains(GUI::Constants::S_FIT_ACTIVITY_PANEL)) {
         settings.beginGroup(GUI::Constants::S_FIT_ACTIVITY_PANEL);
-        m_panelWidth = settings.value(GUI::Constants::S_FIT_ACTIVITY_PANEL_SIZE).toSize().width();
+        setFixedWidth(settings.value(GUI::Constants::S_FIT_ACTIVITY_PANEL_SIZE).toSize().width());
         settings.endGroup();
     }
 }
-
-void JobRealTimeWidget::writeSettings()
-{
-    QSettings settings;
-    settings.beginGroup(GUI::Constants::S_FIT_ACTIVITY_PANEL);
-    settings.setValue(GUI::Constants::S_FIT_ACTIVITY_PANEL_SIZE, size());
-    settings.endGroup();
-    settings.sync();
-}
diff --git a/GUI/View/Fit/JobRealTimeWidget.h b/GUI/View/Fit/JobRealTimeWidget.h
index dba671e8233..d24f0613766 100644
--- a/GUI/View/Fit/JobRealTimeWidget.h
+++ b/GUI/View/Fit/JobRealTimeWidget.h
@@ -28,7 +28,6 @@ class JobRealTimeWidget : public QWidget {
     Q_OBJECT
 public:
     JobRealTimeWidget(JobModel* jobModel, QWidget* parent = nullptr);
-    ~JobRealTimeWidget();
 
     ParameterTuningWidget* parameterTuningWidget(JobItem* jobItem);
 
@@ -37,11 +36,9 @@ public:
 
 private:
     bool isValidJobItem(JobItem* item);
-    void readSettings();
-    void writeSettings();
+    void applySettings();
 
     ItemStackPresenter<ParameterTuningWidget>* m_stackedWidget;
-    int m_panelWidth;
 };
 
 #endif // BORNAGAIN_GUI_VIEW_FIT_JOBREALTIMEWIDGET_H
-- 
GitLab