tempo-plotter Changeset - 12603d48a708 · protofusion repositories

Changeset - 12603d48a708

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Hasan Yavuz ÖZDERYA - 7 years ago 2019-01-12 16:05:03
hy@ozderya.net

fix symbol show settings not loaded at startup

'toggled' is emitted for 'setChecked', 'triggered' is only emitted when user
actually clicks something

1 file changed with 6 insertions and 9 deletions:

src/plotmenu.cpp

0 comments (0 inline, 0 general)

src/plotmenu.cpp

➞

Show inline comments

 /*
-  Copyright © 2017 Hasan Yavuz Özderya
+  Copyright © 2019 Hasan Yavuz Özderya
   This file is part of serialplot.
@@ @@ -66,7 +66,7 @@ PlotMenu::PlotMenu(QWidget* parent) : @@
     setSymbolsMenu.addAction(&setSymbolsAutoAct);
     setSymbolsAutoAct.setCheckable(true);
     setSymbolsAutoAct.setChecked(true);
-    connect(&setSymbolsAutoAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
+    connect(&setSymbolsAutoAct, SELECT<bool>::OVERLOAD_OF(&QAction::toggled),
             [this](bool checked)
+            {
                 if (checked) emit symbolShowChanged(Plot::ShowSymbolsAuto);
@@ @@ -74,18 +74,18 @@ PlotMenu::PlotMenu(QWidget* parent) : @@
     setSymbolsMenu.addAction(&setSymbolsShowAct);
     setSymbolsShowAct.setCheckable(true);
-    connect(&setSymbolsShowAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
+    connect(&setSymbolsShowAct, SELECT<bool>::OVERLOAD_OF(&QAction::toggled),
             [this](bool checked)
+            {
                 if (checked) symbolShowChanged(Plot::ShowSymbolsShow);
+                if (checked) emit symbolShowChanged(Plot::ShowSymbolsShow);
             });
     setSymbolsMenu.addAction(&setSymbolsHideAct);
     setSymbolsHideAct.setCheckable(true);
-    connect(&setSymbolsHideAct, SELECT<bool>::OVERLOAD_OF(&QAction::triggered),
+    connect(&setSymbolsHideAct, SELECT<bool>::OVERLOAD_OF(&QAction::toggled),
             [this](bool checked)
+            {
                 if (checked) symbolShowChanged(Plot::ShowSymbolsHide);
+                if (checked) emit symbolShowChanged(Plot::ShowSymbolsHide);
             });
     // add symbol actions to same group so that they appear as radio buttons
@@ @@ -202,17 +202,14 @@ void PlotMenu::loadSettings(QSettings* s @@
     QString showSymbolsStr = settings->value(SG_Plot_Symbols, QString()).toString();
     if (showSymbolsStr == "auto")
+    {
         // setSymbols(Plot::ShowSymbolsAuto);
         setSymbolsAutoAct.setChecked(true);
+    }
     else if (showSymbolsStr == "show")
+    {
         // setSymbols(Plot::ShowSymbolsShow);
         setSymbolsShowAct.setChecked(true);
+    }
     else if (showSymbolsStr == "hide")
+    {
         // setSymbols(Plot::ShowSymbolsHide);
         setSymbolsHideAct.setChecked(true);
+    }
     else

0 comments (0 inline, 0 general)