tempo-plotter Changeset - c83e2c5aa6fe · protofusion repositories

Changeset - c83e2c5aa6fe

Parent rev.

Child rev.

[Not reviewed]

channelmanager

0 3 0

Hasan Yavuz ÖZDERYA - 9 years ago 2017-01-02 05:36:00
hy@ozderya.net

update color selector on selection change, fix index checks

3 files changed with 31 insertions and 15 deletions:

src/channelinfomodel.cpp

src/channelmanager.cpp

src/plotcontrolpanel.cpp

0 comments (0 inline, 0 general)

src/channelinfomodel.cpp

➞

Show inline comments

@@ @@ -66,7 +66,7 @@ Qt::ItemFlags ChannelInfoModel::flags(co @@
 QVariant ChannelInfoModel::data(const QModelIndex &index, int role) const
+{
     // check index
     if (index.row() >= (int) _numOfChannels)
+    if (index.row() >= (int) _numOfChannels || index.row() < 0)
+    {
         return QVariant();
+    }
@@ @@ -127,7 +127,7 @@ QVariant ChannelInfoModel::headerData(in @@
 bool ChannelInfoModel::setData(const QModelIndex &index, const QVariant &value, int role)
+{
     // check index
     if (index.row() >= (int) _numOfChannels)
+    if (index.row() >= (int) _numOfChannels || index.row() < 0)
+    {
         return false;
+    }
@@ @@ -245,18 +245,20 @@ void ChannelInfoModel::loadSettings(QSet @@
         chanInfo.color      = settings->value(SG_Channels_Color, colors[ci % 8]).value<QColor>();
         chanInfo.visibility = settings->value(SG_Channels_Visible, true).toBool();
-        if ((int) ci >= infos.size())
+        if ((int) ci < infos.size())
+        {
             infos.append(chanInfo);
             infos[ci] = chanInfo;
             if (ci < _numOfChannels)
+            {
                 auto roles = QVector<int>({
                     Qt::DisplayRole, Qt::EditRole, Qt::ForegroundRole, Qt::CheckStateRole});
                 emit dataChanged(index(ci, 0), index(ci, COLUMN_COUNT-1), roles);
+            }
+        }
         else
+        {
             infos[ci] = chanInfo;
             auto roles = QVector<int>({
                     Qt::DisplayRole, Qt::EditRole, Qt::ForegroundRole, Qt::CheckStateRole});
             emit dataChanged(index(ci, 0), index(ci, COLUMN_COUNT-1), roles);
             infos.append(chanInfo);
+        }
+    }

src/channelmanager.cpp

➞

Show inline comments

@@ @@ -140,7 +140,6 @@ void ChannelManager::onChannelInfoChange @@
                 case Qt::EditRole:
                     if (col == ChannelInfoModel::COLUMN_NAME)
+                    {
                         qDebug() << channelName(ci);
                         emit channelNameChanged(ci, channelName(ci));
+                    }
                     break;

src/plotcontrolpanel.cpp

➞

Show inline comments

@@ @@ -210,22 +210,37 @@ void PlotControlPanel::setChannelInfoMod @@
     ui->tvChannelInfo->setModel(model);
     // channel color selector
-    QObject::connect(ui->tvChannelInfo->selectionModel(), &QItemSelectionModel::currentRowChanged,
     connect(ui->tvChannelInfo->selectionModel(), &QItemSelectionModel::currentRowChanged,
                      [this](const QModelIndex &current, const QModelIndex &previous)
+                     {
                          auto model = ui->tvChannelInfo->model();
                          QColor color = model->data(current, Qt::ForegroundRole).value<QColor>();
                          ui->colorSelector->setColor(color);
                          // cpicker.setColor(cim.data(current, Qt::ForegroundRole).value<QColor>());
                      });
-    QObject::connect(ui->colorSelector, &color_widgets::ColorSelector::colorChanged,
     connect(ui->colorSelector, &color_widgets::ColorSelector::colorChanged,
                      [this](QColor color)
+                     {
                          auto index = ui->tvChannelInfo->selectionModel()->currentIndex();
                          // index = index.sibling(index.row(), 0);
                          ui->tvChannelInfo->model()->setData(index, color, Qt::ForegroundRole);
                      });
     connect(model, &QAbstractItemModel::dataChanged,
             [this](const QModelIndex & topLeft, const QModelIndex & bottomRight, const QVector<int> & roles = QVector<int> ())
+            {
                 auto current = ui->tvChannelInfo->selectionModel()->currentIndex();
                 // no current selection
                 if (!current.isValid()) return;
                 auto mod = ui->tvChannelInfo->model();
                 QColor color = mod->data(current, Qt::ForegroundRole).value<QColor>();
                 // temporarily block signals because `setColor` emits `colorChanged`
                 bool wasBlocked = ui->colorSelector->blockSignals(true);
                 ui->colorSelector->setColor(color);
                 ui->colorSelector->blockSignals(wasBlocked);
             });
+}
 void PlotControlPanel::saveSettings(QSettings* settings)

0 comments (0 inline, 0 general)