From 907312f12d5d4059365b9a9ceffed6c523bd62b9 Mon Sep 17 00:00:00 2001
From: Viktor Verebelyi <vviktor2@gmail.com>
Date: Fri, 26 Nov 2021 03:34:11 +0000
Subject: [PATCH] Add selection of graph charts to Frame info window

---
 re/frameinfowindow.cpp |  38 ++++++++++++-
 re/frameinfowindow.h   |   5 ++
 ui/frameinfowindow.ui  | 119 +++++++++++++++++++++++++++++++----------
 3 files changed, 132 insertions(+), 30 deletions(-)

diff --git a/re/frameinfowindow.cpp b/re/frameinfowindow.cpp
index a6508be..b545b50 100644
--- a/re/frameinfowindow.cpp
+++ b/re/frameinfowindow.cpp
@@ -34,6 +34,15 @@ FrameInfoWindow::FrameInfoWindow(const QVector<CANFrame> *frames, QWidget *paren
     connect(MainWindow::getReference(), &MainWindow::framesUpdated, this, &FrameInfoWindow::updatedFrames);
     connect(ui->btnSave, &QAbstractButton::clicked, this, &FrameInfoWindow::saveDetails);
 
+    connect(ui->check_0, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_1, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_2, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_3, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_4, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_5, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_6, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+    connect(ui->check_7, &QCheckBox::stateChanged, this, &FrameInfoWindow::changeGraphVisibility);
+
 
     ui->graphHistogram->setInteractions(QCP::iRangeDrag | QCP::iRangeZoom | QCP::iSelectAxes |
                                     QCP::iSelectLegend | QCP::iSelectPlottables);
@@ -631,7 +640,7 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
         ui->graphBytes->clearGraphs();
         for (int graphs = 0; graphs < 8; graphs++)
         {
-            ui->graphBytes->addGraph();
+            graphRef[graphs] = ui->graphBytes->addGraph();
             ui->graphBytes->graph()->setData(byteGraphX, byteGraphY[graphs]);
             ui->graphBytes->graph()->setPen(bytePens[graphs]);
         }
@@ -652,6 +661,15 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
         ui->timeHistogram->axisRect()->setupFullAxesBox();
         ui->timeHistogram->rescaleAxes();
         ui->timeHistogram->replot();
+
+        ui->check_0->setChecked(true);
+        ui->check_1->setChecked(true);
+        ui->check_2->setChecked(true);
+        ui->check_3->setChecked(true);
+        ui->check_4->setChecked(true);
+        ui->check_5->setChecked(true);
+        ui->check_6->setChecked(true);
+        ui->check_7->setChecked(true);
     }
     else
     {
@@ -726,6 +744,24 @@ void FrameInfoWindow::saveDetails()
     }
 }
 
+void FrameInfoWindow::changeGraphVisibility(int state){
+    QCheckBox *sender = qobject_cast<QCheckBox *>(QObject::sender());
+    if(sender){
+        sender->objectName();
+        int graphId = sender->objectName().right(1).toInt();
+        for (int k = 0; k < 8; k++)
+        {
+            if (k == graphId && graphRef[k] && graphRef[k]->data()){
+                graphRef[k]->setVisible(state);
+                qDebug() << graphId << state << k;
+            }
+        }
+        qDebug() << graphId << state;
+
+        ui->graphBytes->replot();
+    }
+}
+
 void FrameInfoWindow::dumpNode(QTreeWidgetItem* item, QFile *file, int indent)
 {
     for (int i = 0; i < indent; i++) file->write("\t");
diff --git a/re/frameinfowindow.h b/re/frameinfowindow.h
index 72e7a47..5196d93 100644
--- a/re/frameinfowindow.h
+++ b/re/frameinfowindow.h
@@ -9,6 +9,8 @@
 #include "bus_protocols/j1939_handler.h"
 #include "dbc/dbchandler.h"
 
+#include "qcustomplot.h"
+
 namespace Ui {
 class FrameInfoWindow;
 }
@@ -26,6 +28,7 @@ private slots:
     void updateDetailsWindow(QString);
     void updatedFrames(int);
     void saveDetails();
+    void changeGraphVisibility(int state);
 
 private:
     Ui::FrameInfoWindow *ui;
@@ -39,6 +42,8 @@ private:
     static QPen bytePens[8];
     DBCHandler *dbcHandler;
 
+    QCPGraph *graphRef[8];
+
     void refreshIDList();
     void closeEvent(QCloseEvent *event);
     bool eventFilter(QObject *obj, QEvent *event);
diff --git a/ui/frameinfowindow.ui b/ui/frameinfowindow.ui
index 28904e9..406b02a 100644
--- a/ui/frameinfowindow.ui
+++ b/ui/frameinfowindow.ui
@@ -133,7 +133,7 @@
      <item>
       <layout class="QHBoxLayout" name="horizontalLayout_3">
        <item>
-        <widget class="QLabel" name="label_6">
+        <widget class="QCheckBox" name="check_0">
          <property name="palette">
           <palette>
            <active>
@@ -162,9 +162,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -181,10 +181,13 @@
          <property name="text">
           <string>0</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_5">
+        <widget class="QCheckBox" name="check_1">
          <property name="palette">
           <palette>
            <active>
@@ -213,9 +216,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -232,10 +235,50 @@
          <property name="text">
           <string>1</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_9">
+        <widget class="QCheckBox" name="check_2">
+         <property name="palette">
+          <palette>
+           <active>
+            <colorrole role="WindowText">
+             <brush brushstyle="SolidPattern">
+              <color alpha="255">
+               <red>0</red>
+               <green>0</green>
+               <blue>0</blue>
+              </color>
+             </brush>
+            </colorrole>
+           </active>
+           <inactive>
+            <colorrole role="WindowText">
+             <brush brushstyle="SolidPattern">
+              <color alpha="255">
+               <red>0</red>
+               <green>0</green>
+               <blue>0</blue>
+              </color>
+             </brush>
+            </colorrole>
+           </inactive>
+           <disabled>
+            <colorrole role="WindowText">
+             <brush brushstyle="SolidPattern">
+              <color alpha="255">
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
+              </color>
+             </brush>
+            </colorrole>
+           </disabled>
+          </palette>
+         </property>
          <property name="font">
           <font>
            <pointsize>13</pointsize>
@@ -246,10 +289,13 @@
          <property name="text">
           <string>2</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_10">
+        <widget class="QCheckBox" name="check_3">
          <property name="palette">
           <palette>
            <active>
@@ -278,9 +324,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -297,10 +343,13 @@
          <property name="text">
           <string>3</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_7">
+        <widget class="QCheckBox" name="check_4">
          <property name="palette">
           <palette>
            <active>
@@ -329,9 +378,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -348,10 +397,13 @@
          <property name="text">
           <string>4</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_4">
+        <widget class="QCheckBox" name="check_5">
          <property name="palette">
           <palette>
            <active>
@@ -380,9 +432,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -399,10 +451,13 @@
          <property name="text">
           <string>5</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_11">
+        <widget class="QCheckBox" name="check_6">
          <property name="palette">
           <palette>
            <active>
@@ -431,9 +486,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -450,10 +505,13 @@
          <property name="text">
           <string>6</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
        <item>
-        <widget class="QLabel" name="label_8">
+        <widget class="QCheckBox" name="check_7">
          <property name="palette">
           <palette>
            <active>
@@ -482,9 +540,9 @@
             <colorrole role="WindowText">
              <brush brushstyle="SolidPattern">
               <color alpha="255">
-               <red>190</red>
-               <green>190</green>
-               <blue>190</blue>
+               <red>120</red>
+               <green>120</green>
+               <blue>120</blue>
               </color>
              </brush>
             </colorrole>
@@ -501,6 +559,9 @@
          <property name="text">
           <string>7</string>
          </property>
+         <property name="checked">
+          <bool>true</bool>
+         </property>
         </widget>
        </item>
       </layout>