Merge pull request #395 from AdenKoperczak/add_custom_layer_dialog

Fix up custom map inputs, making it easier for others to use
This commit is contained in:
Dan Paulat 2025-03-28 12:23:56 -05:00 committed by GitHub
commit 401dc3ea69
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
14 changed files with 397 additions and 60 deletions

View file

@ -258,6 +258,7 @@ set(HDR_UI source/scwx/qt/ui/about_dialog.hpp
source/scwx/qt/ui/api_key_edit_widget.hpp source/scwx/qt/ui/api_key_edit_widget.hpp
source/scwx/qt/ui/collapsible_group.hpp source/scwx/qt/ui/collapsible_group.hpp
source/scwx/qt/ui/county_dialog.hpp source/scwx/qt/ui/county_dialog.hpp
source/scwx/qt/ui/custom_layer_dialog.hpp
source/scwx/qt/ui/download_dialog.hpp source/scwx/qt/ui/download_dialog.hpp
source/scwx/qt/ui/edit_line_dialog.hpp source/scwx/qt/ui/edit_line_dialog.hpp
source/scwx/qt/ui/edit_marker_dialog.hpp source/scwx/qt/ui/edit_marker_dialog.hpp
@ -290,6 +291,7 @@ set(SRC_UI source/scwx/qt/ui/about_dialog.cpp
source/scwx/qt/ui/api_key_edit_widget.cpp source/scwx/qt/ui/api_key_edit_widget.cpp
source/scwx/qt/ui/collapsible_group.cpp source/scwx/qt/ui/collapsible_group.cpp
source/scwx/qt/ui/county_dialog.cpp source/scwx/qt/ui/county_dialog.cpp
source/scwx/qt/ui/custom_layer_dialog.cpp
source/scwx/qt/ui/download_dialog.cpp source/scwx/qt/ui/download_dialog.cpp
source/scwx/qt/ui/edit_line_dialog.cpp source/scwx/qt/ui/edit_line_dialog.cpp
source/scwx/qt/ui/edit_marker_dialog.cpp source/scwx/qt/ui/edit_marker_dialog.cpp
@ -321,6 +323,7 @@ set(UI_UI source/scwx/qt/ui/about_dialog.ui
source/scwx/qt/ui/animation_dock_widget.ui source/scwx/qt/ui/animation_dock_widget.ui
source/scwx/qt/ui/collapsible_group.ui source/scwx/qt/ui/collapsible_group.ui
source/scwx/qt/ui/county_dialog.ui source/scwx/qt/ui/county_dialog.ui
source/scwx/qt/ui/custom_layer_dialog.ui
source/scwx/qt/ui/edit_line_dialog.ui source/scwx/qt/ui/edit_line_dialog.ui
source/scwx/qt/ui/edit_marker_dialog.ui source/scwx/qt/ui/edit_marker_dialog.ui
source/scwx/qt/ui/gps_info_dialog.ui source/scwx/qt/ui/gps_info_dialog.ui

View file

@ -316,7 +316,7 @@ MainWindow::MainWindow(QWidget* parent) :
p->layerDialog_ = new ui::LayerDialog(this); p->layerDialog_ = new ui::LayerDialog(this);
// Settings Dialog // Settings Dialog
p->settingsDialog_ = new ui::SettingsDialog(this); p->settingsDialog_ = new ui::SettingsDialog(p->settings_, this);
// Map Settings // Map Settings
p->mapSettingsGroup_ = new ui::CollapsibleGroup(tr("Map Settings"), this); p->mapSettingsGroup_ = new ui::CollapsibleGroup(tr("Map Settings"), this);

View file

@ -185,8 +185,6 @@ public:
bool UpdateStoredMapParameters(); bool UpdateStoredMapParameters();
void CheckLevel3Availability(); void CheckLevel3Availability();
std::string FindMapSymbologyLayer();
common::Level2Product common::Level2Product
GetLevel2ProductOrDefault(const std::string& productName) const; GetLevel2ProductOrDefault(const std::string& productName) const;
@ -1146,43 +1144,6 @@ void MapWidget::DumpLayerList() const
logger_->info("Layers: {}", p->map_->layerIds().join(", ").toStdString()); logger_->info("Layers: {}", p->map_->layerIds().join(", ").toStdString());
} }
std::string MapWidgetImpl::FindMapSymbologyLayer()
{
std::string before = "ferry";
for (const QString& qlayer : styleLayers_)
{
const std::string layer = qlayer.toStdString();
// Draw below layers defined in map style
auto it = std::find_if(currentStyle_->drawBelow_.cbegin(),
currentStyle_->drawBelow_.cend(),
[&layer](const std::string& styleLayer) -> bool
{
// Perform case-insensitive matching
RE2 re {"(?i)" + styleLayer};
if (re.ok())
{
return RE2::FullMatch(layer, re);
}
else
{
// Fall back to basic comparison if RE
// doesn't compile
return layer == styleLayer;
}
});
if (it != currentStyle_->drawBelow_.cend())
{
before = layer;
break;
}
}
return before;
}
void MapWidgetImpl::AddLayers() void MapWidgetImpl::AddLayers()
{ {
if (styleLayers_.isEmpty()) if (styleLayers_.isEmpty())
@ -1218,7 +1179,8 @@ void MapWidgetImpl::AddLayers()
{ {
// Subsequent layers are drawn underneath the map symbology layer // Subsequent layers are drawn underneath the map symbology layer
case types::MapLayer::MapUnderlay: case types::MapLayer::MapUnderlay:
before = FindMapSymbologyLayer(); before = util::maplibre::FindMapSymbologyLayer(
styleLayers_, currentStyle_->drawBelow_);
break; break;
// Subsequent layers are drawn after all style-defined layers // Subsequent layers are drawn after all style-defined layers

View file

@ -107,6 +107,10 @@ public:
SCWX_SETTINGS_ENUM_VALIDATOR(scwx::util::ClockFormat, SCWX_SETTINGS_ENUM_VALIDATOR(scwx::util::ClockFormat,
scwx::util::ClockFormatIterator(), scwx::util::ClockFormatIterator(),
scwx::util::GetClockFormatName)); scwx::util::GetClockFormatName));
customStyleUrl_.SetValidator(
[](const std::string& value)
{ return value.find("key=") == std::string::npos; });
customStyleDrawLayer_.SetValidator([](const std::string& value) customStyleDrawLayer_.SetValidator([](const std::string& value)
{ return !value.empty(); }); { return !value.empty(); });
defaultAlertAction_.SetValidator( defaultAlertAction_.SetValidator(

View file

@ -12,6 +12,7 @@
#include <QLineEdit> #include <QLineEdit>
#include <QSpinBox> #include <QSpinBox>
#include <QWidget> #include <QWidget>
#include <utility>
#include <vector> #include <vector>
namespace scwx::qt::settings namespace scwx::qt::settings
@ -19,6 +20,9 @@ namespace scwx::qt::settings
static const std::string logPrefix_ = "scwx::qt::settings::settings_interface"; static const std::string logPrefix_ = "scwx::qt::settings::settings_interface";
static const QString kValidStyleSheet_ = "";
static const QString kInvalidStyleSheet_ = "border: 2px solid red;";
template<class T> template<class T>
class SettingsInterface<T>::Impl class SettingsInterface<T>::Impl
{ {
@ -40,6 +44,7 @@ public:
void UpdateEditWidget(); void UpdateEditWidget();
void UpdateResetButton(); void UpdateResetButton();
void UpdateUnitLabel(); void UpdateUnitLabel();
void UpdateValidityDisplay();
SettingsInterface<T>* self_; SettingsInterface<T>* self_;
@ -59,6 +64,8 @@ public:
bool unitEnabled_ {false}; bool unitEnabled_ {false};
bool trimmingEnabled_ {false}; bool trimmingEnabled_ {false};
std::optional<std::string> invalidTooltip_;
}; };
template<class T> template<class T>
@ -172,14 +179,13 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
p->context_.get(), p->context_.get(),
[this](const QKeySequence& sequence) [this](const QKeySequence& sequence)
{ {
std::string value { const std::string value {
sequence.toString().toStdString()}; sequence.toString().toStdString()};
// Attempt to stage the value // Attempt to stage the value
p->stagedValid_ = p->variable_->StageValue(value); p->stagedValid_ = p->variable_->StageValue(value);
p->UpdateResetButton(); p->UpdateResetButton();
p->UpdateValidityDisplay();
// TODO: Display invalid status
}); });
} }
} }
@ -194,7 +200,7 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
p->context_.get(), p->context_.get(),
[this](const QString& text) [this](const QString& text)
{ {
QString trimmedText = const QString trimmedText =
p->trimmingEnabled_ ? text.trimmed() : text; p->trimmingEnabled_ ? text.trimmed() : text;
// Map to value if required // Map to value if required
@ -207,8 +213,7 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
// Attempt to stage the value // Attempt to stage the value
p->stagedValid_ = p->variable_->StageValue(value); p->stagedValid_ = p->variable_->StageValue(value);
p->UpdateResetButton(); p->UpdateResetButton();
p->UpdateValidityDisplay();
// TODO: Display invalid status
}); });
} }
else if constexpr (std::is_same_v<T, double>) else if constexpr (std::is_same_v<T, double>)
@ -221,8 +226,8 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
[this](const QString& text) [this](const QString& text)
{ {
// Convert to a double // Convert to a double
bool ok; bool ok = false;
double value = text.toDouble(&ok); const double value = text.toDouble(&ok);
if (ok) if (ok)
{ {
// Attempt to stage the value // Attempt to stage the value
@ -235,6 +240,8 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
p->stagedValid_ = false; p->stagedValid_ = false;
p->UpdateResetButton(); p->UpdateResetButton();
} }
p->UpdateValidityDisplay();
}); });
} }
else if constexpr (std::is_same_v<T, std::vector<std::int64_t>>) else if constexpr (std::is_same_v<T, std::vector<std::int64_t>>)
@ -279,8 +286,7 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
// Attempt to stage the value // Attempt to stage the value
p->stagedValid_ = p->variable_->StageValue(value); p->stagedValid_ = p->variable_->StageValue(value);
p->UpdateResetButton(); p->UpdateResetButton();
p->UpdateValidityDisplay();
// TODO: Display invalid status
}); });
} }
} }
@ -364,6 +370,8 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
p->UpdateResetButton(); p->UpdateResetButton();
} }
// Otherwise, don't process an unchanged value // Otherwise, don't process an unchanged value
p->UpdateValidityDisplay();
}); });
} }
} }
@ -415,6 +423,8 @@ void SettingsInterface<T>::SetEditWidget(QWidget* widget)
p->UpdateResetButton(); p->UpdateResetButton();
} }
// Otherwise, don't process an unchanged value // Otherwise, don't process an unchanged value
p->UpdateValidityDisplay();
}); });
} }
} }
@ -500,6 +510,13 @@ void SettingsInterface<T>::EnableTrimming(bool trimmingEnabled)
p->trimmingEnabled_ = trimmingEnabled; p->trimmingEnabled_ = trimmingEnabled;
} }
template<class T>
void SettingsInterface<T>::SetInvalidTooltip(
const std::optional<std::string>& tooltip)
{
p->invalidTooltip_ = std::move(tooltip);
}
template<class T> template<class T>
template<class U> template<class U>
void SettingsInterface<T>::Impl::SetWidgetText(U* widget, const T& currentValue) void SettingsInterface<T>::Impl::SetWidgetText(U* widget, const T& currentValue)
@ -617,6 +634,15 @@ void SettingsInterface<T>::Impl::UpdateUnitLabel()
unitLabel_->setText(QString::fromStdString(unitAbbreviation_.value_or(""))); unitLabel_->setText(QString::fromStdString(unitAbbreviation_.value_or("")));
} }
template<class T>
void SettingsInterface<T>::Impl::UpdateValidityDisplay()
{
editWidget_->setStyleSheet(stagedValid_ ? kValidStyleSheet_ :
kInvalidStyleSheet_);
editWidget_->setToolTip(
invalidTooltip_ && !stagedValid_ ? invalidTooltip_->c_str() : "");
}
template<class T> template<class T>
void SettingsInterface<T>::Impl::UpdateResetButton() void SettingsInterface<T>::Impl::UpdateResetButton()
{ {

View file

@ -1,5 +1,6 @@
#pragma once #pragma once
#include <optional>
#include <scwx/qt/settings/settings_interface_base.hpp> #include <scwx/qt/settings/settings_interface_base.hpp>
#include <functional> #include <functional>
@ -130,6 +131,13 @@ public:
*/ */
void EnableTrimming(bool trimmingEnabled = true); void EnableTrimming(bool trimmingEnabled = true);
/**
* Set a tooltip to be displayed when an invalid input is given.
*
* @param tooltip the tooltip to be displayed
*/
void SetInvalidTooltip(const std::optional<std::string>& tooltip);
private: private:
class Impl; class Impl;
std::unique_ptr<Impl> p; std::unique_ptr<Impl> p;

View file

@ -0,0 +1,108 @@
#include "custom_layer_dialog.hpp"
#include "ui_custom_layer_dialog.h"
#include <scwx/qt/settings/general_settings.hpp>
#include <scwx/qt/util/maplibre.hpp>
#include <scwx/util/logger.hpp>
#include <scwx/qt/map/map_provider.hpp>
#include <utility>
namespace scwx::qt::ui
{
static const std::string logPrefix_ = "scwx::qt::ui::custom_layer_dialog";
static const auto logger_ = scwx::util::Logger::Create(logPrefix_);
class CustomLayerDialogImpl
{
public:
explicit CustomLayerDialogImpl(CustomLayerDialog* self,
QMapLibre::Settings settings) :
self_(self), settings_(std::move(settings))
{
}
~CustomLayerDialogImpl() = default;
CustomLayerDialogImpl(const CustomLayerDialogImpl&) = delete;
CustomLayerDialogImpl(CustomLayerDialogImpl&&) = delete;
CustomLayerDialogImpl& operator=(const CustomLayerDialogImpl&) = delete;
CustomLayerDialogImpl& operator=(CustomLayerDialogImpl&&) = delete;
void handle_mapChanged(QMapLibre::Map::MapChange change);
CustomLayerDialog* self_;
QMapLibre::Settings settings_;
std::shared_ptr<QMapLibre::Map> map_;
};
void CustomLayerDialogImpl::handle_mapChanged(QMapLibre::Map::MapChange change)
{
if (change == QMapLibre::Map::MapChange::MapChangeDidFinishLoadingStyle)
{
auto& generalSettings = settings::GeneralSettings::Instance();
const std::string& customStyleDrawLayer =
generalSettings.custom_style_draw_layer().GetStagedOrValue();
const QStringList layerIds = map_->layerIds();
self_->ui->layerListWidget->clear();
self_->ui->layerListWidget->addItems(layerIds);
const std::string symbologyLayer = util::maplibre::FindMapSymbologyLayer(
layerIds, {customStyleDrawLayer});
const auto& symbologyItems = self_->ui->layerListWidget->findItems(
symbologyLayer.c_str(), Qt::MatchExactly);
if (!symbologyItems.isEmpty())
{
self_->ui->layerListWidget->setCurrentItem(symbologyItems.first());
}
}
}
CustomLayerDialog::CustomLayerDialog(const QMapLibre::Settings& settings,
QWidget* parent) :
QDialog(parent),
p {std::make_unique<CustomLayerDialogImpl>(this, settings)},
ui(new Ui::CustomLayerDialog)
{
ui->setupUi(this);
auto& generalSettings = settings::GeneralSettings::Instance();
const auto& customStyleUrl = generalSettings.custom_style_url().GetValue();
const auto mapProvider =
map::GetMapProvider(generalSettings.map_provider().GetValue());
// TODO render the map with a layer to show what they are selecting
p->map_ = std::make_shared<QMapLibre::Map>(
nullptr, p->settings_, QSize(1, 1), devicePixelRatioF());
QString qUrl = QString::fromStdString(customStyleUrl);
if (mapProvider == map::MapProvider::MapTiler)
{
qUrl.append("?key=");
qUrl.append(map::GetMapProviderApiKey(mapProvider));
}
p->map_->setStyleUrl(qUrl);
QObject::connect(p->map_.get(),
&QMapLibre::Map::mapChanged,
this,
[this](QMapLibre::Map::MapChange change)
{ p->handle_mapChanged(change); });
}
CustomLayerDialog::~CustomLayerDialog()
{
delete ui;
}
std::string CustomLayerDialog::selected_layer()
{
return ui->layerListWidget->currentItem()->text().toStdString();
}
} // namespace scwx::qt::ui

View file

@ -0,0 +1,35 @@
#pragma once
#include <qmaplibre.hpp>
#include <QDialog>
#include <string>
namespace Ui
{
class CustomLayerDialog;
}
namespace scwx::qt::ui
{
class CustomLayerDialogImpl;
class CustomLayerDialog : public QDialog
{
Q_OBJECT
Q_DISABLE_COPY_MOVE(CustomLayerDialog)
public:
explicit CustomLayerDialog(const QMapLibre::Settings& settings,
QWidget* parent = nullptr);
~CustomLayerDialog() override;
std::string selected_layer();
private:
friend class CustomLayerDialogImpl;
std::unique_ptr<CustomLayerDialogImpl> p;
Ui::CustomLayerDialog* ui;
};
} // namespace scwx::qt::ui

View file

@ -0,0 +1,96 @@
<?xml version="1.0" encoding="UTF-8"?>
<ui version="4.0">
<class>CustomLayerDialog</class>
<widget class="QDialog" name="CustomLayerDialog">
<property name="geometry">
<rect>
<x>0</x>
<y>0</y>
<width>308</width>
<height>300</height>
</rect>
</property>
<property name="sizePolicy">
<sizepolicy hsizetype="Minimum" vsizetype="Minimum">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
<property name="windowTitle">
<string>Custom Map Style Draw Layer</string>
</property>
<layout class="QVBoxLayout" name="verticalLayout">
<item>
<layout class="QHBoxLayout" name="horizontalLayout">
<item>
<widget class="QListWidget" name="layerListWidget">
<property name="sizePolicy">
<sizepolicy hsizetype="Preferred" vsizetype="Expanding">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
<property name="editTriggers">
<set>QAbstractItemView::EditTrigger::NoEditTriggers</set>
</property>
<property name="alternatingRowColors">
<bool>true</bool>
</property>
</widget>
</item>
</layout>
</item>
<item>
<widget class="QDialogButtonBox" name="buttonBox">
<property name="sizePolicy">
<sizepolicy hsizetype="Minimum" vsizetype="Fixed">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
<property name="orientation">
<enum>Qt::Orientation::Horizontal</enum>
</property>
<property name="standardButtons">
<set>QDialogButtonBox::StandardButton::Cancel|QDialogButtonBox::StandardButton::Ok</set>
</property>
</widget>
</item>
</layout>
</widget>
<resources/>
<connections>
<connection>
<sender>buttonBox</sender>
<signal>accepted()</signal>
<receiver>CustomLayerDialog</receiver>
<slot>accept()</slot>
<hints>
<hint type="sourcelabel">
<x>248</x>
<y>254</y>
</hint>
<hint type="destinationlabel">
<x>157</x>
<y>274</y>
</hint>
</hints>
</connection>
<connection>
<sender>buttonBox</sender>
<signal>rejected()</signal>
<receiver>CustomLayerDialog</receiver>
<slot>reject()</slot>
<hints>
<hint type="sourcelabel">
<x>316</x>
<y>260</y>
</hint>
<hint type="destinationlabel">
<x>286</x>
<y>274</y>
</hint>
</hints>
</connection>
</connections>
</ui>

View file

@ -24,6 +24,7 @@
#include <scwx/qt/types/time_types.hpp> #include <scwx/qt/types/time_types.hpp>
#include <scwx/qt/types/unit_types.hpp> #include <scwx/qt/types/unit_types.hpp>
#include <scwx/qt/ui/county_dialog.hpp> #include <scwx/qt/ui/county_dialog.hpp>
#include <scwx/qt/ui/custom_layer_dialog.hpp>
#include <scwx/qt/ui/radar_site_dialog.hpp> #include <scwx/qt/ui/radar_site_dialog.hpp>
#include <scwx/qt/ui/serial_port_dialog.hpp> #include <scwx/qt/ui/serial_port_dialog.hpp>
#include <scwx/qt/ui/settings/alert_palette_settings_widget.hpp> #include <scwx/qt/ui/settings/alert_palette_settings_widget.hpp>
@ -45,6 +46,7 @@
#include <QPushButton> #include <QPushButton>
#include <QStandardItemModel> #include <QStandardItemModel>
#include <QToolButton> #include <QToolButton>
#include <utility>
namespace scwx namespace scwx
{ {
@ -106,7 +108,8 @@ static const std::unordered_map<std::string, ColorTableConversions>
class SettingsDialogImpl class SettingsDialogImpl
{ {
public: public:
explicit SettingsDialogImpl(SettingsDialog* self) : explicit SettingsDialogImpl(SettingsDialog* self,
QMapLibre::Settings& mapSettings) :
self_ {self}, self_ {self},
radarSiteDialog_ {new RadarSiteDialog(self)}, radarSiteDialog_ {new RadarSiteDialog(self)},
alertAudioRadarSiteDialog_ {new RadarSiteDialog(self)}, alertAudioRadarSiteDialog_ {new RadarSiteDialog(self)},
@ -114,6 +117,7 @@ public:
countyDialog_ {new CountyDialog(self)}, countyDialog_ {new CountyDialog(self)},
wfoDialog_ {new WFODialog(self)}, wfoDialog_ {new WFODialog(self)},
fontDialog_ {new QFontDialog(self)}, fontDialog_ {new QFontDialog(self)},
mapSettings_ {mapSettings},
fontCategoryModel_ {new QStandardItemModel(self)}, fontCategoryModel_ {new QStandardItemModel(self)},
settings_ {std::initializer_list<settings::SettingsInterfaceBase*> { settings_ {std::initializer_list<settings::SettingsInterfaceBase*> {
&defaultRadarSite_, &defaultRadarSite_,
@ -219,6 +223,8 @@ public:
WFODialog* wfoDialog_; WFODialog* wfoDialog_;
QFontDialog* fontDialog_; QFontDialog* fontDialog_;
QMapLibre::Settings& mapSettings_;
QStandardItemModel* fontCategoryModel_; QStandardItemModel* fontCategoryModel_;
types::FontCategory selectedFontCategory_ {types::FontCategory::Unknown}; types::FontCategory selectedFontCategory_ {types::FontCategory::Unknown};
@ -292,9 +298,10 @@ public:
std::vector<settings::SettingsInterfaceBase*> settings_; std::vector<settings::SettingsInterfaceBase*> settings_;
}; };
SettingsDialog::SettingsDialog(QWidget* parent) : SettingsDialog::SettingsDialog(QMapLibre::Settings& mapSettings,
QWidget* parent) :
QDialog(parent), QDialog(parent),
p {std::make_unique<SettingsDialogImpl>(this)}, p {std::make_unique<SettingsDialogImpl>(this, mapSettings)},
ui(new Ui::SettingsDialog) ui(new Ui::SettingsDialog)
{ {
ui->setupUi(this); ui->setupUi(this);
@ -685,12 +692,39 @@ void SettingsDialogImpl::SetupGeneralTab()
customStyleUrl_.SetSettingsVariable(generalSettings.custom_style_url()); customStyleUrl_.SetSettingsVariable(generalSettings.custom_style_url());
customStyleUrl_.SetEditWidget(self_->ui->customMapUrlLineEdit); customStyleUrl_.SetEditWidget(self_->ui->customMapUrlLineEdit);
customStyleUrl_.SetResetButton(self_->ui->resetCustomMapUrlButton); customStyleUrl_.SetResetButton(self_->ui->resetCustomMapUrlButton);
customStyleUrl_.SetInvalidTooltip(
"Remove anything following \"?key=\" in the URL");
customStyleUrl_.EnableTrimming(); customStyleUrl_.EnableTrimming();
customStyleDrawLayer_.SetSettingsVariable( customStyleDrawLayer_.SetSettingsVariable(
generalSettings.custom_style_draw_layer()); generalSettings.custom_style_draw_layer());
customStyleDrawLayer_.SetEditWidget(self_->ui->customMapLayerLineEdit); customStyleDrawLayer_.SetEditWidget(self_->ui->customMapLayerLineEdit);
customStyleDrawLayer_.SetResetButton(self_->ui->resetCustomMapLayerButton); customStyleDrawLayer_.SetResetButton(self_->ui->resetCustomMapLayerButton);
QObject::connect(
self_->ui->customMapLayerToolButton,
&QAbstractButton::clicked,
self_,
[this]()
{
// WA_DeleteOnClose manages memory
// NOLINTNEXTLINE(cppcoreguidelines-owning-memory)
auto* customLayerDialog = new ui::CustomLayerDialog(mapSettings_);
customLayerDialog->setAttribute(Qt::WA_DeleteOnClose);
QObject::connect(
customLayerDialog,
&QDialog::accepted,
self_,
[this, customLayerDialog]()
{
auto newLayer = customLayerDialog->selected_layer();
self_->ui->customMapLayerLineEdit->setText(newLayer.c_str());
// setText does not emit the textEdited signal
Q_EMIT
self_->ui->customMapLayerLineEdit->textEdited(newLayer.c_str());
});
customLayerDialog->open();
});
defaultAlertAction_.SetSettingsVariable( defaultAlertAction_.SetSettingsVariable(
generalSettings.default_alert_action()); generalSettings.default_alert_action());

View file

@ -1,6 +1,7 @@
#pragma once #pragma once
#include <QDialog> #include <QDialog>
#include <qmaplibre.hpp>
namespace Ui namespace Ui
{ {
@ -24,7 +25,8 @@ private:
Q_DISABLE_COPY(SettingsDialog) Q_DISABLE_COPY(SettingsDialog)
public: public:
explicit SettingsDialog(QWidget* parent = nullptr); explicit SettingsDialog(QMapLibre::Settings& mapSettings,
QWidget* parent = nullptr);
~SettingsDialog(); ~SettingsDialog();
private: private:

View file

@ -122,7 +122,7 @@
</sizepolicy> </sizepolicy>
</property> </property>
<property name="currentIndex"> <property name="currentIndex">
<number>3</number> <number>0</number>
</property> </property>
<widget class="QWidget" name="general"> <widget class="QWidget" name="general">
<layout class="QVBoxLayout" name="verticalLayout_2"> <layout class="QVBoxLayout" name="verticalLayout_2">
@ -135,9 +135,9 @@
<property name="geometry"> <property name="geometry">
<rect> <rect>
<x>0</x> <x>0</x>
<y>-303</y> <y>-260</y>
<width>511</width> <width>511</width>
<height>733</height> <height>812</height>
</rect> </rect>
</property> </property>
<layout class="QVBoxLayout" name="verticalLayout"> <layout class="QVBoxLayout" name="verticalLayout">
@ -608,6 +608,13 @@
</property> </property>
</widget> </widget>
</item> </item>
<item row="15" column="3">
<widget class="QToolButton" name="customMapLayerToolButton">
<property name="text">
<string>...</string>
</property>
</widget>
</item>
</layout> </layout>
</widget> </widget>
</item> </item>
@ -714,8 +721,8 @@
<rect> <rect>
<x>0</x> <x>0</x>
<y>0</y> <y>0</y>
<width>98</width> <width>80</width>
<height>28</height> <height>18</height>
</rect> </rect>
</property> </property>
<layout class="QGridLayout" name="gridLayout_3"> <layout class="QGridLayout" name="gridLayout_3">

View file

@ -1,7 +1,9 @@
#include <scwx/qt/util/maplibre.hpp> #include <scwx/qt/util/maplibre.hpp>
#include <QMapLibre/Utils> #include <QMapLibre/Utils>
#include <algorithm>
#include <mbgl/util/constants.hpp> #include <mbgl/util/constants.hpp>
#include <re2/re2.h>
namespace scwx namespace scwx
{ {
@ -120,6 +122,44 @@ void SetMapStyleUrl(const std::shared_ptr<map::MapContext>& mapContext,
} }
} }
std::string FindMapSymbologyLayer(const QStringList& styleLayers,
const std::vector<std::string>& drawBelow)
{
std::string before = "ferry";
for (const QString& qlayer : styleLayers)
{
const std::string layer = qlayer.toStdString();
// Draw below layers defined in map style
auto it =
std::ranges::find_if(drawBelow,
[&layer](const std::string& styleLayer) -> bool
{
// Perform case-insensitive matching
const RE2 re {"(?i)" + styleLayer};
if (re.ok())
{
return RE2::FullMatch(layer, re);
}
else
{
// Fall back to basic comparison if RE
// doesn't compile
return layer == styleLayer;
}
});
if (it != drawBelow.cend())
{
before = layer;
break;
}
}
return before;
}
} // namespace maplibre } // namespace maplibre
} // namespace util } // namespace util
} // namespace qt } // namespace qt

View file

@ -37,6 +37,18 @@ glm::vec2 LatLongToScreenCoordinate(const QMapLibre::Coordinate& coordinate);
void SetMapStyleUrl(const std::shared_ptr<map::MapContext>& mapContext, void SetMapStyleUrl(const std::shared_ptr<map::MapContext>& mapContext,
const std::string& url); const std::string& url);
/**
* @brief Find the first layer which should be drawn above the radar products
*
* @param [in] styleLayers The layers of the style
* @param [in] drawBelow A list of RE2 compatible regex's describing the layers
* to draw below
*
* @return The first layer to be drawn above the radar products
*/
std::string FindMapSymbologyLayer(const QStringList& styleLayers,
const std::vector<std::string>& drawBelow);
} // namespace maplibre } // namespace maplibre
} // namespace util } // namespace util
} // namespace qt } // namespace qt