Compare commits

...

2 commits

Author SHA1 Message Date
Jan Käberich bbd1bce0b8 accept enter key along with return key when setting values
Some checks failed
Build / PC_Application_Ubuntu (push) Has been cancelled
Build / PC_Application_RPi5 (push) Has been cancelled
Build / PC_Application_Windows (push) Has been cancelled
Build / PC_Application_OSX (push) Has been cancelled
Build / PC_Application_OSX_13 (push) Has been cancelled
Build / Embedded_Firmware (push) Has been cancelled
HIL_Tests / Get_Repository (push) Has been cancelled
Unit_Tests / Tests (push) Has been cancelled
HIL_Tests / PC_Application_RPi5 (push) Has been cancelled
HIL_Tests / Embedded_Firmware (push) Has been cancelled
HIL_Tests / HIL (push) Has been cancelled
2025-11-05 10:41:56 +01:00
Jan Käberich 937b002dfb Minor UI improvememts
- Add space between number and unit
- Increase width of toolbar items to (hopefully) be fully visible for
  all operating systems and themes
- Adjust order of automatically created calibration measurements to
  match LibreCAL
- Disable edit triggers for calibration measurement table
- Show timestamps of calibration measurements in local time
- Allow starting calibration measurements by double clicking row
2025-11-05 09:19:02 +01:00
5 changed files with 17 additions and 7 deletions

View file

@ -533,8 +533,9 @@ void Calibration::edit(TraceModel *traceModel)
ui->table->setCellWidget(i, 1, measurements[i]->createStandardWidget()); ui->table->setCellWidget(i, 1, measurements[i]->createStandardWidget());
ui->table->setCellWidget(i, 2, measurements[i]->createSettingsWidget()); ui->table->setCellWidget(i, 2, measurements[i]->createSettingsWidget());
ui->table->setItem(i, 3, measurements[i]->getStatisticsItem()); ui->table->setItem(i, 3, measurements[i]->getStatisticsItem());
ui->table->setItem(i, 4, new QTableWidgetItem(measurements[i]->getTimestamp().toString())); ui->table->setItem(i, 4, new QTableWidgetItem(measurements[i]->getTimestamp().toLocalTime().toString()));
} }
ui->table->setEditTriggers(QAbstractItemView::NoEditTriggers);
ui->table->selectRow(row); ui->table->selectRow(row);
updateTableEditButtons(); updateTableEditButtons();
}; };
@ -625,6 +626,11 @@ void Calibration::edit(TraceModel *traceModel)
emit startMeasurements(m); emit startMeasurements(m);
}); });
// double clicking on a row also starts the measurement
connect(ui->table, &QTableWidget::doubleClicked, this, [=](){
emit ui->measure->clicked();
});
connect(ui->selectMeasurement, &QPushButton::clicked, [=](){ connect(ui->selectMeasurement, &QPushButton::clicked, [=](){
auto selected = ui->table->selectionModel()->selectedRows(); auto selected = ui->table->selectionModel()->selectedRows();
if(selected.size() != 1) { if(selected.size() != 1) {
@ -1971,12 +1977,12 @@ void Calibration::createDefaultMeasurements(Calibration::DefaultMeasurements dm)
{ {
lock_guard<recursive_mutex> guard(access); lock_guard<recursive_mutex> guard(access);
auto createSOL = [=](int port) { auto createSOL = [=](int port) {
auto _short = new CalibrationMeasurement::Short(this);
_short->setPort(port);
measurements.push_back(_short);
auto open = new CalibrationMeasurement::Open(this); auto open = new CalibrationMeasurement::Open(this);
open->setPort(port); open->setPort(port);
measurements.push_back(open); measurements.push_back(open);
auto _short = new CalibrationMeasurement::Short(this);
_short->setPort(port);
measurements.push_back(_short);
auto load = new CalibrationMeasurement::Load(this); auto load = new CalibrationMeasurement::Load(this);
load->setPort(port); load->setPort(port);
measurements.push_back(load); measurements.push_back(load);

View file

@ -59,7 +59,7 @@ bool SIUnitEdit::eventFilter(QObject *, QEvent *event)
clearFocus(); clearFocus();
return true; return true;
} }
if(key == Qt::Key_Return) { if(key == Qt::Key_Return || key == Qt::Key_Enter) {
// use new value without prefix // use new value without prefix
parseNewValue(1.0); parseNewValue(1.0);
continueEditing(); continueEditing();

View file

@ -112,7 +112,7 @@ SpectrumAnalyzer::SpectrumAnalyzer(AppWindow *window, QString name)
auto eStart = new SIUnitEdit("Hz", " kMG", 6); auto eStart = new SIUnitEdit("Hz", " kMG", 6);
// calculate width required with expected string length // calculate width required with expected string length
auto width = QFontMetrics(eStart->font()).horizontalAdvance("3.00000GHz") + 15; auto width = QFontMetrics(eStart->font()).horizontalAdvance("10.00000 MHz") + 15;
eStart->setFixedWidth(width); eStart->setFixedWidth(width);
eStart->setToolTip("Start frequency"); eStart->setToolTip("Start frequency");
connect(eStart, &SIUnitEdit::valueChanged, this, &SpectrumAnalyzer::SetStartFreq); connect(eStart, &SIUnitEdit::valueChanged, this, &SpectrumAnalyzer::SetStartFreq);

View file

@ -325,7 +325,7 @@ VNA::VNA(AppWindow *window, QString name)
auto eStart = new SIUnitEdit("Hz", " kMG", 6); auto eStart = new SIUnitEdit("Hz", " kMG", 6);
// calculate width required with expected string length // calculate width required with expected string length
auto width = QFontMetrics(eStart->font()).horizontalAdvance("3.00000GHz") + 15; auto width = QFontMetrics(eStart->font()).horizontalAdvance("10.00000 MHz") + 15;
eStart->setFixedWidth(width); eStart->setFixedWidth(width);
eStart->setToolTip("Start frequency"); eStart->setToolTip("Start frequency");
connect(eStart, &SIUnitEdit::valueChanged, this, &VNA::SetStartFreq); connect(eStart, &SIUnitEdit::valueChanged, this, &VNA::SetStartFreq);

View file

@ -68,6 +68,10 @@ QString Unit::ToString(double value, QString unit, QString prefixes, int precisi
} }
ss << value; ss << value;
sValue.append(QString::fromStdString(ss.str())); sValue.append(QString::fromStdString(ss.str()));
if(!unit.isEmpty() && !unit.startsWith(' ')) {
// add space between number and unit
sValue.append(' ');
}
sValue.append(prefixes[prefixIndex]); sValue.append(prefixes[prefixIndex]);
} }
sValue.append(unit); sValue.append(unit);