Settings: Prettify the settings dialog

[pulseview.git] / pv / dialogs / settings.cpp

diff --git a/pv/dialogs/settings.cpp b/pv/dialogs/settings.cpp
index 8cb8d67de31ad58958464bb1f27e72afa665cddd..c29d80b68f9ddc7d434041e4ef915f92f14c8470 100644 (file)
--- a/pv/dialogs/settings.cpp
+++ b/pv/dialogs/settings.cpp
@@ -24,27 +24,58 @@
 #include <QDialogButtonBox>
 #include <QFormLayout>
 #include <QGroupBox>
-#include <QTabWidget>
+#include <QHBoxLayout>
 #include <QVBoxLayout>
 
 namespace pv {
 namespace dialogs {
 
 Settings::Settings(QWidget *parent) :
-       QDialog(parent, 0)
+       QDialog(parent, nullptr)
 {
-       QTabWidget *tab_stack = new QTabWidget(this);
-       tab_stack->addTab(get_view_settings_form(tab_stack), tr("&Views"));
+       const int icon_size = 64;
+
+       page_list = new QListWidget;
+       page_list->setViewMode(QListView::IconMode);
+       page_list->setIconSize(QSize(icon_size, icon_size));
+       page_list->setMovement(QListView::Static);
+       page_list->setMaximumWidth(icon_size + icon_size/2);
+       page_list->setSpacing(12);
+
+       pages = new QStackedWidget;
+       create_pages();
+
+       QHBoxLayout *tab_layout = new QHBoxLayout;
+       tab_layout->addWidget(page_list);
+       tab_layout->addWidget(pages, Qt::AlignLeft);
 
        QDialogButtonBox *button_box = new QDialogButtonBox(
                QDialogButtonBox::Ok | QDialogButtonBox::Cancel);
 
        QVBoxLayout* root_layout = new QVBoxLayout(this);
-       root_layout->addWidget(tab_stack);
+       root_layout->addLayout(tab_layout);
        root_layout->addWidget(button_box);
 
        connect(button_box, SIGNAL(accepted()), this, SLOT(accept()));
        connect(button_box, SIGNAL(rejected()), this, SLOT(reject()));
+       connect(page_list, SIGNAL(currentItemChanged(QListWidgetItem*, QListWidgetItem*)),
+               this, SLOT(on_page_changed(QListWidgetItem*, QListWidgetItem*)));
+
+       // Start to record changes
+       GlobalSettings settings;
+       settings.start_tracking();
+}
+
+void Settings::create_pages()
+{
+       // View page
+       pages->addWidget(get_view_settings_form(pages));
+
+       QListWidgetItem *viewButton = new QListWidgetItem(page_list);
+       viewButton->setIcon(QIcon(":/icons/sigrok-logo-notext.svg"));
+       viewButton->setText(tr("Views"));
+       viewButton->setTextAlignment(Qt::AlignHCenter);
+       viewButton->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled);
 }
 
 QWidget *Settings::get_view_settings_form(QWidget *parent) const
@@ -76,14 +107,28 @@ QWidget *Settings::get_view_settings_form(QWidget *parent) const
 
 void Settings::accept()
 {
+       GlobalSettings settings;
+       settings.stop_tracking();
+
        QDialog::accept();
 }
 
 void Settings::reject()
 {
+       GlobalSettings settings;
+       settings.undo_tracked_changes();
+
        QDialog::reject();
 }
 
+void Settings::on_page_changed(QListWidgetItem *current, QListWidgetItem *previous)
+{
+       if (!current)
+               current = previous;
+
+       pages->setCurrentIndex(page_list->row(current));
+}
+
 void Settings::on_view_alwaysZoomToFit_changed(int state)
 {
        GlobalSettings settings;