Change connection type for MouseCoordinateChanged slot to queued to avoid recursive paint

scwx-qt/source/scwx/qt/main/main_window.cpp

@@ -745,7 +745,8 @@ void MainWindowImpl::ConnectMapSignals()
                  }
               });
 
-      connect(mapWidget,
+      connect(
+         mapWidget,
          &map::MapWidget::MouseCoordinateChanged,
          this,
          [this](common::Coordinate coordinate)
@@ -758,7 +759,8 @@ void MainWindowImpl::ConnectMapSignals()
             coordinateLabel_->setText(
                QString("%1, %2").arg(latitude).arg(longitude));
             coordinateLabel_->setVisible(true);
-              });
+         },
+         Qt::QueuedConnection);
 
       connect(
          mapWidget,

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

@@ -148,7 +148,17 @@ signals:
                              double bearing,
                              double pitch);
    void MapStyleChanged(const std::string& styleName);
+
+   /**
+    * This signal is emitted when the mouse moves to a different geographic
+    * coordinate within the map widget. This signal is emitted during paintGL(),
+    * and must be connected using a connection type of queued if the slot
+    * triggers a repaint.
+    *
+    * @param [in] coordinate Geographic coordinate of the mouse
+    */
    void MouseCoordinateChanged(common::Coordinate coordinate);
+
    void RadarSiteRequested(const std::string& id);
    void RadarSiteUpdated(std::shared_ptr<config::RadarSite> radarSite);
    void RadarSweepUpdated();