From 6a4aa86600f2f1633507018adb7ec307aa96e758 2017-07-27 14:39:40
From: Alexandre Leroux <aleroux@itlink.fr>
Date: 2017-07-27 14:39:40
Subject: [PATCH] Creates settings dialog and adds it to a new menu in MainWindow

The dialog will contain a set of widgets saved as a list (one entry per widget).

By selecting an entry in the list, the corresponding widget will be displayed

---

diff --git a/app/src/MainWindow.cpp b/app/src/MainWindow.cpp
index b2f2078..221121c 100644
--- a/app/src/MainWindow.cpp
+++ b/app/src/MainWindow.cpp
@@ -24,6 +24,7 @@
 
 #include <DataSource/DataSourceController.h>
 #include <DataSource/DataSourceWidget.h>
+#include <Settings/SqpSettingsDialog.h>
 #include <SidePane/SqpSidePane.h>
 #include <SqpApplication.h>
 #include <Time/TimeController.h>
@@ -55,14 +56,23 @@ const auto RIGHTMAININSPECTORWIDGETSPLITTERINDEX = 4;
 
 class MainWindow::MainWindowPrivate {
 public:
+    explicit MainWindowPrivate(MainWindow *mainWindow)
+            : m_LastOpenLeftInspectorSize{},
+              m_LastOpenRightInspectorSize{},
+              m_SettingsDialog{new SqpSettingsDialog{mainWindow}}
+    {
+    }
+
     QSize m_LastOpenLeftInspectorSize;
     QSize m_LastOpenRightInspectorSize;
+    /// Settings dialog. MainWindow has the ownership
+    SqpSettingsDialog *m_SettingsDialog;
 };
 
 MainWindow::MainWindow(QWidget *parent)
         : QMainWindow{parent},
           m_Ui{new Ui::MainWindow},
-          impl{spimpl::make_unique_impl<MainWindowPrivate>()}
+          impl{spimpl::make_unique_impl<MainWindowPrivate>(this)}
 {
     m_Ui->setupUi(this);
 
@@ -155,12 +165,24 @@ MainWindow::MainWindow(QWidget *parent)
                 openInspector(checked, true, openRightInspectorAction);
             });
 
+    // //// //
+    // Menu //
+    // //// //
     this->menuBar()->addAction(tr("File"));
+    auto toolsMenu = this->menuBar()->addMenu(tr("Tools"));
+    toolsMenu->addAction(tr("Settings..."), [this]() {
+        impl->m_SettingsDialog->exec();
+    });
+
     auto mainToolBar = this->addToolBar(QStringLiteral("MainToolBar"));
 
     auto timeWidget = new TimeWidget{};
     mainToolBar->addWidget(timeWidget);
 
+    // /////////// //
+    // Connections //
+    // /////////// //
+
     // Controllers / controllers connections
     connect(&sqpApp->timeController(), SIGNAL(timeUpdated(SqpDateTime)),
             &sqpApp->variableController(), SLOT(onDateTimeOnSelection(SqpDateTime)));
diff --git a/gui/include/Settings/SqpSettingsDialog.h b/gui/include/Settings/SqpSettingsDialog.h
new file mode 100644
index 0000000..4ee6c2f
--- /dev/null
+++ b/gui/include/Settings/SqpSettingsDialog.h
@@ -0,0 +1,27 @@
+#ifndef SCIQLOP_SQPSETTINGSDIALOG_H
+#define SCIQLOP_SQPSETTINGSDIALOG_H
+
+#include "Settings/ISqpSettingsBindable.h"
+
+#include <QDialog>
+
+namespace Ui {
+class SqpSettingsDialog;
+} // Ui
+
+/**
+ * @brief The SqpSettingsDialog class represents the dialog in which the parameters of SciQlop are
+ * set
+ */
+class SqpSettingsDialog : public QDialog {
+    Q_OBJECT
+
+public:
+    explicit SqpSettingsDialog(QWidget *parent = 0);
+    virtual ~SqpSettingsDialog() noexcept;
+
+private:
+    Ui::SqpSettingsDialog *ui;
+};
+
+#endif // SCIQLOP_SQPSETTINGSDIALOG_H
diff --git a/gui/src/Settings/SqpSettingsDialog.cpp b/gui/src/Settings/SqpSettingsDialog.cpp
new file mode 100644
index 0000000..7b923c2
--- /dev/null
+++ b/gui/src/Settings/SqpSettingsDialog.cpp
@@ -0,0 +1,17 @@
+#include "Settings/SqpSettingsDialog.h"
+#include "ui_SqpSettingsDialog.h"
+
+SqpSettingsDialog::SqpSettingsDialog(QWidget *parent)
+        : QDialog{parent}, ui{new Ui::SqpSettingsDialog}
+{
+    ui->setupUi(this);
+
+    // Connection to change the current page to the selection of an entry in the list
+    connect(ui->listWidget, &QListWidget::currentRowChanged, ui->stackedWidget,
+            &QStackedWidget::setCurrentIndex);
+}
+
+SqpSettingsDialog::~SqpSettingsDialog() noexcept
+{
+    delete ui;
+}
diff --git a/gui/ui/Settings/SqpSettingsDialog.ui b/gui/ui/Settings/SqpSettingsDialog.ui
new file mode 100644
index 0000000..e75b37c
--- /dev/null
+++ b/gui/ui/Settings/SqpSettingsDialog.ui
@@ -0,0 +1,84 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
+ <class>SqpSettingsDialog</class>
+ <widget class="QDialog" name="SqpSettingsDialog">
+  <property name="geometry">
+   <rect>
+    <x>0</x>
+    <y>0</y>
+    <width>572</width>
+    <height>394</height>
+   </rect>
+  </property>
+  <property name="windowTitle">
+   <string>SciQlop Settings</string>
+  </property>
+  <layout class="QGridLayout" name="gridLayout">
+   <item row="0" column="0">
+    <widget class="QListWidget" name="listWidget">
+     <property name="sizePolicy">
+      <sizepolicy hsizetype="Minimum" vsizetype="Expanding">
+       <horstretch>0</horstretch>
+       <verstretch>0</verstretch>
+      </sizepolicy>
+     </property>
+    </widget>
+   </item>
+   <item row="0" column="1">
+    <widget class="QStackedWidget" name="stackedWidget">
+     <property name="sizePolicy">
+      <sizepolicy hsizetype="Expanding" vsizetype="Preferred">
+       <horstretch>0</horstretch>
+       <verstretch>0</verstretch>
+      </sizepolicy>
+     </property>
+    </widget>
+   </item>
+   <item row="1" column="0" colspan="2">
+    <widget class="QDialogButtonBox" name="buttonBox">
+     <property name="orientation">
+      <enum>Qt::Horizontal</enum>
+     </property>
+     <property name="standardButtons">
+      <set>QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
+     </property>
+    </widget>
+   </item>
+  </layout>
+ </widget>
+ <resources/>
+ <connections>
+  <connection>
+   <sender>buttonBox</sender>
+   <signal>accepted()</signal>
+   <receiver>SqpSettingsDialog</receiver>
+   <slot>accept()</slot>
+   <hints>
+    <hint type="sourcelabel">
+     <x>248</x>
+     <y>254</y>
+    </hint>
+    <hint type="destinationlabel">
+     <x>157</x>
+     <y>274</y>
+    </hint>
+   </hints>
+  </connection>
+  <connection>
+   <sender>buttonBox</sender>
+   <signal>rejected()</signal>
+   <receiver>SqpSettingsDialog</receiver>
+   <slot>reject()</slot>
+   <hints>
+    <hint type="sourcelabel">
+     <x>316</x>
+     <y>260</y>
+    </hint>
+    <hint type="destinationlabel">
+     <x>286</x>
+     <y>274</y>
+    </hint>
+   </hints>
+  </connection>
+ </connections>
+</ui>