Provide visual feedback tracking is replaying position

Author: nirvn

src/core/tracker.cpp

@@ -350,13 +350,6 @@ void Tracker::replayPositionInformationList( const QList<GnssPositionInformation
 {
   const qint64 startTime = QDateTime::currentMSecsSinceEpoch();
 
-  bool wasActive = false;
-  if ( mIsActive )
-  {
-    wasActive = true;
-    stop();
-  }
-
   mIsReplaying = true;
   emit isReplayingChanged();
 
@@ -396,15 +389,27 @@ void Tracker::replayPositionInformationList( const QList<GnssPositionInformation
   mIsReplaying = false;
   emit isReplayingChanged();
 
-  if ( wasActive )
+  if ( mIsSuspended )
   {
+    mIsSuspended = false;
+    emit isSuspendedChanged();
     start();
   }
 
   const qint64 endTime = QDateTime::currentMSecsSinceEpoch();
   qInfo() << QStringLiteral( "Tracker position information replay duration: %1ms" ).arg( endTime - startTime );
 }
 
+void Tracker::suspendUntilReplay()
+{
+  if ( mIsActive )
+  {
+    mIsSuspended = true;
+    emit isSuspendedChanged();
+    stop();
+  }
+}
+
 void Tracker::rubberbandModelVertexCountChanged()
 {
   if ( ( !mIsActive && !mIsReplaying ) || mRubberbandModel->vertexCount() == 0 )

src/core/tracker.h

@@ -35,6 +35,7 @@ class Tracker : public QObject
     Q_OBJECT
 
     Q_PROPERTY( bool isActive READ isActive NOTIFY isActiveChanged )
+    Q_PROPERTY( bool isSuspended READ isSuspended NOTIFY isSuspendedChanged )
     Q_PROPERTY( bool isReplaying READ isReplaying NOTIFY isReplayingChanged )
 
     Q_PROPERTY( bool visible READ visible WRITE setVisible NOTIFY visibleChanged )
@@ -134,6 +135,9 @@ class Tracker : public QObject
     //! Returns whether the tracker has been started
     bool isActive() const { return mIsActive; }
 
+    //! Returns whether the track has been suspended
+    bool isSuspended() const { return mIsSuspended; }
+
     //! Returns whether the tracker is replaying positions
     bool isReplaying() const { return mIsReplaying; }
 
@@ -148,8 +152,11 @@ class Tracker : public QObject
     //! Replays a list of position information taking into account the tracker settings
     void replayPositionInformationList( const QList<GnssPositionInformation> &positionInformationList, QgsQuickCoordinateTransformer *coordinateTransformer = nullptr );
 
+    void suspendUntilReplay();
+
   signals:
     void isActiveChanged();
+    void isSuspendedChanged();
     void isReplayingChanged();
     void visibleChanged();
     void vectorLayerChanged();
@@ -177,6 +184,7 @@ class Tracker : public QObject
     void trackPosition();
 
     bool mIsActive = false;
+    bool mIsSuspended = false;
     bool mIsReplaying = false;
 
     RubberbandModel *mRubberbandModel = nullptr;

src/core/trackingmodel.cpp

@@ -169,13 +169,25 @@ void TrackingModel::replayPositionInformationList( const QList<GnssPositionInfor
   for ( int i = 0; i < mTrackers.size(); i++ )
   {
     Tracker *tracker = mTrackers[i];
-    if ( tracker->isActive() )
+    if ( tracker->isSuspended() )
     {
       tracker->replayPositionInformationList( positionInformationList, coordinateTransformer );
     }
   }
 }
 
+void TrackingModel::suspendUntilReplay()
+{
+  for ( int i = 0; i < mTrackers.size(); i++ )
+  {
+    Tracker *tracker = mTrackers[i];
+    if ( tracker->isActive() )
+    {
+      tracker->suspendUntilReplay();
+    }
+  }
+}
+
 void TrackingModel::setTrackerVisibility( QgsVectorLayer *layer, bool visible )
 {
   if ( trackerIterator( layer ) != mTrackers.constEnd() )

src/core/trackingmodel.h

@@ -71,6 +71,8 @@ class TrackingModel : public QAbstractItemModel
     //! Replays a list of position information for all active trackers
     Q_INVOKABLE void replayPositionInformationList( const QList<GnssPositionInformation> &positionInformationList, QgsQuickCoordinateTransformer *coordinateTransformer = nullptr );
 
+    Q_INVOKABLE void suspendUntilReplay();
+
     void reset();
 
     /**

src/qml/TrackingSession.qml

@@ -23,10 +23,8 @@ Item {
     enabled: tracker.isActive
 
     function onPositionInformationChanged() {
-      if (!positionSource.backgroundMode) {
-        featureModel.positionInformation = positionSource.positionInformation;
-        tracker.processPositionInformation(positionSource.positionInformation, positionSource.projectedPosition);
-      }
+      featureModel.positionInformation = positionSource.positionInformation;
+      tracker.processPositionInformation(positionSource.positionInformation, positionSource.projectedPosition);
     }
   }
 

src/qml/qgismobileapp.qml

@@ -351,17 +351,34 @@ ApplicationWindow {
     }
 
     onBackgroundModeChanged: {
-      if (!backgroundMode) {
-        mapCanvasMap.freeze('trackerreplay');
-        let list = positionSource.getBackgroundPositionInformation();
-        // Qt bug weirdly returns an empty list on first invokation to source, call twice to insure we've got the actual list
-        list = positionSource.getBackgroundPositionInformation();
-        trackingModel.replayPositionInformationList(list, coordinateTransformer);
-        mapCanvasMap.unfreeze('trackerreplay');
+      if (trackings.count > 0) {
+        if (backgroundMode) {
+          trackingModel.suspendUntilReplay();
+        } else {
+          busyOverlay.text = qsTr("Replaying collected positions, hold on");
+          busyOverlay.state = "visible";
+          replayTimer.restart();
+        }
       }
     }
   }
 
+  Timer {
+    id: replayTimer
+
+    interval: 250
+    repeat: false
+    onTriggered: {
+      mapCanvasMap.freeze('trackerreplay');
+      let list = positionSource.getBackgroundPositionInformation();
+      // Qt bug weirdly returns an empty list on first invokation to source, call twice to insure we've got the actual list
+      list = positionSource.getBackgroundPositionInformation();
+      trackingModel.replayPositionInformationList(list, positionSource.coordinateTransformer);
+      mapCanvasMap.unfreeze('trackerreplay');
+      busyOverlay.state = "hidden";
+    }
+  }
+
   PositioningSettings {
     id: positioningSettings
     objectName: "positioningSettings"