diff --git a/Software/PC_Application/Traces/markerwidget.cpp b/Software/PC_Application/Traces/markerwidget.cpp index 8bb2d11..8f68f68 100644 --- a/Software/PC_Application/Traces/markerwidget.cpp +++ b/Software/PC_Application/Traces/markerwidget.cpp @@ -39,13 +39,12 @@ MarkerWidget::~MarkerWidget() void MarkerWidget::on_bDelete_clicked() { if (model.rowCount() <= 0) { - return; // prevent crash if bDelete clicked with no markers (empty model) + return; // there is nothing to delete (empty model) } QModelIndex ind = ui->treeView->currentIndex(); if ( ! ind.isValid() ) { - return; // add marker(s), then click bDelete without clicking on any marker (there is no index clicked in treeView) - // alternative: select last marker, then proceede to delete it? + return; // if no marker clicked/selected in treeView, the index is not valid } auto marker = model.markerFromIndex(ind); diff --git a/Software/PC_Application/Traces/tracemarker.cpp b/Software/PC_Application/Traces/tracemarker.cpp index df692b7..1453807 100644 --- a/Software/PC_Application/Traces/tracemarker.cpp +++ b/Software/PC_Application/Traces/tracemarker.cpp @@ -768,6 +768,7 @@ SIUnitEdit *TraceMarker::getSettingsEditor() case Type::Lowpass: case Type::Highpass: case Type::PeakTable: + case Type::Bandpass: // initialize with "dB" return new SIUnitEdit("db", " ", 3); case Type::TOI: return nullptr;