[FIX] fixed the show box plot button toggling

saif-el · Feb 11, 2021 · 9c84f04 · 9c84f04
1 parent bf9862d
commit 9c84f04
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 6 deletions.
diff --git a/src/gui/mzroll/eicwidget.cpp b/src/gui/mzroll/eicwidget.cpp
@@ -1784,6 +1784,11 @@ void EicWidget::showBarPlot(bool f) {
     optionBarPlotChecked(f);
 }
 
+void EicWidget::showBoxPlot(bool f) {
+	_showBoxPlot = f;
+    optionBoxPlotChecked(f);
+}
+
 void EicWidget::contextMenuEvent(QContextMenuEvent * event)
 {
 	event->ignore();

diff --git a/src/gui/mzroll/eicwidget.h b/src/gui/mzroll/eicwidget.h
@@ -137,9 +137,7 @@ public Q_SLOTS:
 
 	void showBarPlot(bool f);
 
-	void showBoxPlot(bool f) {
-		_showBoxPlot = f;
-	}
+	void showBoxPlot(bool f);
 
 	void autoZoom(bool f) {
 		_autoZoom = f;
@@ -191,6 +189,7 @@ public Q_SLOTS:
     void compoundSet(Compound*);
 	void optionTicChecked(bool);
 	void optionBarPlotChecked(bool);
+	void optionBoxPlotChecked(bool);
 
 private:
 	EICLogic* eicParameters;

diff --git a/src/gui/mzroll/mainwindow.cpp b/src/gui/mzroll/mainwindow.cpp
@@ -4247,9 +4247,11 @@ QWidget* MainWindowWidgetAction::createWidget(QWidget *parent) {
         btnShowBoxplot->setToolTip(tr("Show box-plot"));
 		btnShowBoxplot->setCheckable(true);
 		btnShowBoxplot->setChecked(false);
-		connect(btnShowBoxplot,SIGNAL(toggled(bool)),  mw->getEicWidget(),SLOT(showBoxPlot(bool)));
-		connect(btnShowBoxplot,SIGNAL(toggled(bool)),mw,SLOT(analyticsBoxPlot()));
-		connect(btnShowBoxplot,SIGNAL(toggled(bool)), mw->getEicWidget(), SLOT(replot()));
+		connect(btnShowBoxplot, &QToolButton::toggled, mw->getEicWidget(), &EicWidget::showBoxPlot);
+		connect(btnShowBoxplot, SIGNAL(toggled(bool)), mw, SLOT(analyticsBoxPlot()));
+		connect(btnShowBoxplot, SIGNAL(toggled(bool)), mw->getEicWidget(), SLOT(replot()));
+        connect(mw->getEicWidget(), &EicWidget::optionBoxPlotChecked, btnShowBoxplot, &QToolButton::setChecked);
+
 		return btnShowBoxplot;
 	}
     else if (btnName == "spacer") {