diff --git a/apps/OpenSpace/ext/launcher/src/profile/horizonsdialog.cpp b/apps/OpenSpace/ext/launcher/src/profile/horizonsdialog.cpp index b1cb9df20e..d247b1011e 100644 --- a/apps/OpenSpace/ext/launcher/src/profile/horizonsdialog.cpp +++ b/apps/OpenSpace/ext/launcher/src/profile/horizonsdialog.cpp @@ -24,6 +24,7 @@ #include "profile/horizonsdialog.h" +#include #include "profile/line.h" #include #include @@ -46,6 +47,8 @@ #include namespace { + constexpr const char* _loggerCat = "HorizonsDialog"; + // Text for the different units for the step size constexpr const char* TimeVarying = "arcseconds (time-varying)"; constexpr const char* Minutes = "minutes"; @@ -438,8 +441,8 @@ bool HorizonsDialog::isValidInput() { std::to_string(std::numeric_limits::max()); } else { - // In the case of arcseconds (first option) range is different - if (_timeTypeCombo->currentIndex() == 0) { + // In the case of arcseconds range is different + if (_timeTypeCombo->currentText() == TimeVarying) { if (60 > step || step > 3600) { isValid = false; message = "Angular step size needs to be in range 60 to 3600"; @@ -532,7 +535,7 @@ std::string HorizonsDialog::constructUrl() { unit = ""; } else { - _errorMsg->setText("Invalid unit type"); + _errorMsg->setText("Invalid time unit type"); return ""; } @@ -885,9 +888,11 @@ void HorizonsDialog::appendLog(const std::string& message, const LogLevel level) { case HorizonsDialog::LogLevel::Error: htmlText = "(E) "; + LERROR(message); break; case HorizonsDialog::LogLevel::Warning: htmlText = "(W) "; + LWARNING(message); break; case HorizonsDialog::LogLevel::Info: htmlText = "(I) ";