Add geographic coordinates to mouse picking functions

scwx-qt/source/scwx/qt/gl/draw/draw_item.cpp

@@ -57,7 +57,8 @@ bool DrawItem::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& /* params */,
    const QPointF& /* mouseLocalPos */,
    const QPointF& /* mouseGlobalPos */,
-   const glm::vec2& /* mouseCoords */)
+   const glm::vec2& /* mouseCoords */,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    // By default, the draw item is not picked
    return false;

scwx-qt/source/scwx/qt/gl/draw/draw_item.hpp

@@ -1,6 +1,7 @@
 #pragma once
 
 #include <scwx/qt/gl/gl.hpp>
+#include <scwx/common/geographic.hpp>
 
 #include <memory>
 
@@ -41,14 +42,16 @@ public:
     * @param [in] mouseLocalPos Mouse cursor widget position
     * @param [in] mouseGlobalPos Mouse cursor screen position
     * @param [in] mouseCoords Mouse cursor location in map screen coordinates
+    * @param [in] mouseGeoCoords Mouse cursor location in geographic coordinates
     *
     * @return true if the draw item was picked, otherwise false
     */
    virtual bool
    RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                   const QPointF&   mouseLocalPos,
-                   const QPointF&   mouseGlobalPos,
-                   const glm::vec2& mouseCoords);
+                   const QPointF&            mouseLocalPos,
+                   const QPointF&            mouseGlobalPos,
+                   const glm::vec2&          mouseCoords,
+                   const common::Coordinate& mouseGeoCoords);
 
 protected:
    void

scwx-qt/source/scwx/qt/gl/draw/geo_icons.cpp

@@ -794,7 +794,8 @@ bool GeoIcons::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& params,
    const QPointF& /* mouseLocalPos */,
    const QPointF&   mouseGlobalPos,
-   const glm::vec2& mouseCoords)
+   const glm::vec2& mouseCoords,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    std::unique_lock lock {p->iconMutex_};
 

scwx-qt/source/scwx/qt/gl/draw/geo_icons.hpp

@@ -38,9 +38,10 @@ public:
    void Deinitialize() override;
 
    bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override;
+                        const QPointF&            mouseLocalPos,
+                        const QPointF&            mouseGlobalPos,
+                        const glm::vec2&          mouseCoords,
+                        const common::Coordinate& mouseGeoCoords) override;
 
    /**
     * Sets the visibility of the geo icons.

scwx-qt/source/scwx/qt/gl/draw/placefile_icons.cpp

@@ -688,7 +688,8 @@ bool PlacefileIcons::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& params,
    const QPointF& /* mouseLocalPos */,
    const QPointF&   mouseGlobalPos,
-   const glm::vec2& mouseCoords)
+   const glm::vec2& mouseCoords,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    std::unique_lock lock {p->iconMutex_};
 

scwx-qt/source/scwx/qt/gl/draw/placefile_icons.hpp

@@ -36,9 +36,10 @@ public:
    void Deinitialize() override;
 
    bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override;
+                        const QPointF&            mouseLocalPos,
+                        const QPointF&            mouseGlobalPos,
+                        const glm::vec2&          mouseCoords,
+                        const common::Coordinate& mouseGeoCoords) override;
 
    /**
     * Resets and prepares the draw item for adding a new set of icons.

scwx-qt/source/scwx/qt/gl/draw/placefile_lines.cpp

@@ -499,7 +499,8 @@ bool PlacefileLines::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& params,
    const QPointF& /* mouseLocalPos */,
    const QPointF&   mouseGlobalPos,
-   const glm::vec2& mouseCoords)
+   const glm::vec2& mouseCoords,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    std::unique_lock lock {p->lineMutex_};
 

scwx-qt/source/scwx/qt/gl/draw/placefile_lines.hpp

@@ -33,9 +33,10 @@ public:
    void Deinitialize() override;
 
    bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override;
+                        const QPointF&            mouseLocalPos,
+                        const QPointF&            mouseGlobalPos,
+                        const glm::vec2&          mouseCoords,
+                        const common::Coordinate& mouseGeoCoords) override;
 
    /**
     * Resets and prepares the draw item for adding a new set of lines.

scwx-qt/source/scwx/qt/gl/draw/placefile_text.cpp

@@ -241,7 +241,8 @@ bool PlacefileText::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& /* params */,
    const QPointF& /* mouseLocalPos */,
    const QPointF& mouseGlobalPos,
-   const glm::vec2& /* mouseCoords */)
+   const glm::vec2& /* mouseCoords */,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    bool itemPicked = false;
 

scwx-qt/source/scwx/qt/gl/draw/placefile_text.hpp

@@ -38,9 +38,10 @@ public:
    void Deinitialize() override;
 
    bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override;
+                        const QPointF&            mouseLocalPos,
+                        const QPointF&            mouseGlobalPos,
+                        const glm::vec2&          mouseCoords,
+                        const common::Coordinate& mouseGeoCoords) override;
 
    /**
     * Resets and prepares the draw item for adding a new set of text.

scwx-qt/source/scwx/qt/map/draw_layer.cpp

@@ -80,7 +80,8 @@ bool DrawLayer::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& params,
    const QPointF&                                  mouseLocalPos,
    const QPointF&                                  mouseGlobalPos,
-   const glm::vec2&                                mouseCoords)
+   const glm::vec2&                                mouseCoords,
+   const common::Coordinate&                       mouseGeoCoords)
 {
    bool itemPicked = false;
 
@@ -88,8 +89,11 @@ bool DrawLayer::RunMousePicking(
    for (auto it = p->drawList_.rbegin(); it != p->drawList_.rend(); ++it)
    {
       // Run mouse picking on each draw item
-      if ((*it)->RunMousePicking(
-             params, mouseLocalPos, mouseGlobalPos, mouseCoords))
+      if ((*it)->RunMousePicking(params,
+                                 mouseLocalPos,
+                                 mouseGlobalPos,
+                                 mouseCoords,
+                                 mouseGeoCoords))
       {
          // If a draw item was picked, don't process additional items
          itemPicked = true;

scwx-qt/source/scwx/qt/map/draw_layer.hpp

@@ -25,9 +25,10 @@ public:
 
    virtual bool
    RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                   const QPointF&   mouseLocalPos,
-                   const QPointF&   mouseGlobalPos,
-                   const glm::vec2& mouseCoords) override;
+                   const QPointF&            mouseLocalPos,
+                   const QPointF&            mouseGlobalPos,
+                   const glm::vec2&          mouseCoords,
+                   const common::Coordinate& mouseGeoCoords) override;
 
 protected:
    void AddDrawItem(const std::shared_ptr<gl::draw::DrawItem>& drawItem);

scwx-qt/source/scwx/qt/map/generic_layer.cpp

@@ -30,7 +30,8 @@ bool GenericLayer::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& /* params */,
    const QPointF& /* mouseLocalPos */,
    const QPointF& /* mouseGlobalPos */,
-   const glm::vec2& /* mousePos */)
+   const glm::vec2& /* mousePos */,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    // By default, the layer has nothing to pick
    return false;

scwx-qt/source/scwx/qt/map/generic_layer.hpp

@@ -1,6 +1,7 @@
 #pragma once
 
 #include <scwx/qt/map/map_context.hpp>
+#include <scwx/common/geographic.hpp>
 
 #include <memory>
 
@@ -36,14 +37,16 @@ public:
     * @param [in] mouseLocalPos Mouse cursor widget position
     * @param [in] mouseGlobalPos Mouse cursor screen position
     * @param [in] mouseCoords Mouse cursor location in map screen coordinates
+    * @param [in] mouseGeoCoords Mouse cursor location in geographic coordinates
     *
     * @return true if a draw item was picked, otherwise false
     */
    virtual bool
    RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                   const QPointF&   mouseLocalPos,
-                   const QPointF&   mouseGlobalPos,
-                   const glm::vec2& mouseCoords);
+                   const QPointF&            mouseLocalPos,
+                   const QPointF&            mouseGlobalPos,
+                   const glm::vec2&          mouseCoords,
+                   const common::Coordinate& mouseGeoCoords);
 
 signals:
    void NeedsRendering();

scwx-qt/source/scwx/qt/map/map_widget.cpp

@@ -1215,8 +1215,11 @@ void MapWidgetImpl::RunMousePicking()
    for (auto it = genericLayers_.rbegin(); it != genericLayers_.rend(); ++it)
    {
       // Run mouse picking for each layer
-      if ((*it)->RunMousePicking(
-             params, lastPos_, lastGlobalPos_, mouseScreenCoordinate))
+      if ((*it)->RunMousePicking(params,
+                                 lastPos_,
+                                 lastGlobalPos_,
+                                 mouseScreenCoordinate,
+                                 {coordinate.first, coordinate.second}))
       {
          // If a draw item was picked, don't process additional layers
          itemPicked = true;

scwx-qt/source/scwx/qt/map/overlay_layer.cpp

@@ -277,7 +277,8 @@ bool OverlayLayer::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& /* params */,
    const QPointF& /* mouseLocalPos */,
    const QPointF& /* mouseGlobalPos */,
-   const glm::vec2& /* mouseCoords */)
+   const glm::vec2& /* mouseCoords */,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    // If sweep time was picked, don't process additional items
    return p->sweepTimePicked_;

scwx-qt/source/scwx/qt/map/overlay_layer.hpp

@@ -21,10 +21,12 @@ public:
    void Render(const QMapLibreGL::CustomLayerRenderParameters&) override final;
    void Deinitialize() override final;
 
-   bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override final;
+   bool
+   RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
+                   const QPointF&            mouseLocalPos,
+                   const QPointF&            mouseGlobalPos,
+                   const glm::vec2&          mouseCoords,
+                   const common::Coordinate& mouseGeoCoords) override final;
 
 public slots:
    void UpdateSweepTimeNextFrame();

scwx-qt/source/scwx/qt/map/radar_site_layer.cpp

@@ -157,7 +157,8 @@ bool RadarSiteLayer::RunMousePicking(
    const QMapLibreGL::CustomLayerRenderParameters& /* params */,
    const QPointF& /* mouseLocalPos */,
    const QPointF& mouseGlobalPos,
-   const glm::vec2& /* mouseCoords */)
+   const glm::vec2& /* mouseCoords */,
+   const common::Coordinate& /* mouseGeoCoords */)
 {
    if (!p->hoverText_.empty())
    {

scwx-qt/source/scwx/qt/map/radar_site_layer.hpp

@@ -22,10 +22,12 @@ public:
    void Render(const QMapLibreGL::CustomLayerRenderParameters&) override final;
    void Deinitialize() override final;
 
-   bool RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
-                        const QPointF&   mouseLocalPos,
-                        const QPointF&   mouseGlobalPos,
-                        const glm::vec2& mouseCoords) override final;
+   bool
+   RunMousePicking(const QMapLibreGL::CustomLayerRenderParameters& params,
+                   const QPointF&            mouseLocalPos,
+                   const QPointF&            mouseGlobalPos,
+                   const glm::vec2&          mouseCoords,
+                   const common::Coordinate& mouseGeoCoords) override final;
 
 signals:
    void RadarSiteSelected(const std::string& id);