Merge pull request #22 from nbgsmk/master

Bandpass marker: init editor with "dB"

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);

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;