New initial map position handling

Also show QGC position on map

src/FlightDisplay/FlightDisplayView.qml

@@ -50,7 +50,6 @@ Item {
 
     property var _activeVehicle:  multiVehicleManager.activeVehicle
 
-    readonly property var  _defaultVehicleCoordinate:   mainWindow.tabletPosition
     readonly property real _defaultRoll:                0
     readonly property real _defaultPitch:               0
     readonly property real _defaultHeading:             0
@@ -71,8 +70,6 @@ Item {
     property real _pitch:               _activeVehicle ? (isNaN(_activeVehicle.pitch)   ? _defaultPitch   : _activeVehicle.pitch)   : _defaultPitch
     property real _heading:             _activeVehicle ? (isNaN(_activeVehicle.heading) ? _defaultHeading : _activeVehicle.heading) : _defaultHeading
 
-    property var  _vehicleCoordinate:   _activeVehicle ? (_activeVehicle.coordinateValid ? _activeVehicle.coordinate : _defaultVehicleCoordinate) : _defaultVehicleCoordinate
-
     property real _altitudeWGS84:       _activeVehicle ? _activeVehicle.altitudeWGS84 : _defaultAltitudeWGS84
     property real _groundSpeed:         _activeVehicle ? _activeVehicle.groundSpeed   : _defaultGroundSpeed
     property real _airSpeed:            _activeVehicle ? _activeVehicle.airSpeed      : _defaultAirSpeed
@@ -120,7 +117,6 @@ Item {
                     _flightMap.zoomLevel = _savedZoomLevel
                 else
                     _savedZoomLevel = _flightMap.zoomLevel
-                _flightMap.updateMapPosition(true /* force */)
             } else {
                 _flightVideo = item
             }

src/FlightDisplay/FlightDisplayViewMap.qml

@@ -39,18 +39,15 @@ FlightMap {
     id:             flightMap
     anchors.fill:   parent
     mapName:        _mapName
-    latitude:       mainWindow.tabletPosition.latitude
-    longitude:      mainWindow.tabletPosition.longitude
 
-    property var    rootVehicleCoordinate:  _vehicleCoordinate
-    property bool   _followVehicle:         true
+    property bool   _followVehicle:                 true
+    property bool   _activeVehicleCoordinateValid:  multiVehicleManager.activeVehicle ? multiVehicleManager.activeVehicle.coordinateValid : false
+    property var    activeVehicleCoordinate:        multiVehicleManager.activeVehicle ? multiVehicleManager.activeVehicle.coordinate : QtPositioning.coordinate()
 
-    onRootVehicleCoordinateChanged: updateMapPosition(false /* force */)
-
-    function updateMapPosition(force) {
-        if (_followVehicle || force) {
-            flightMap.latitude  = root._vehicleCoordinate.latitude
-            flightMap.longitude = root._vehicleCoordinate.longitude
+    onActiveVehicleCoordinateChanged: {
+        if (_followVehicle && activeVehicleCoordinate.isValid) {
+            _initialMapPositionSet = true
+            flightMap.center  = activeVehicleCoordinate
         }
     }
 

src/FlightMap/FlightMap.qml

@@ -43,22 +43,16 @@ import QGroundControl.Mavlink               1.0
 Map {
     id: _map
 
-    property real   latitude:           64.154549   //-- If you find yourself here on startup, something went wrong :)
-    property real   longitude:          -22.023540
-    property real   heading:            0
     property bool   interactive:        true
     property string mapName:            'defaultMap'
     property string mapType:            QGroundControl.flightMapSettings.mapTypeForMapName(mapName)
 //  property alias  mapWidgets:         controlWidgets
     property bool   isSatelliteMap:     mapType == "Satellite Map" || mapType == "Hybrid Map"
 
-    property real   lon: (longitude >= -180 && longitude <= 180) ? longitude : 0
-    property real   lat: (latitude  >=  -90 && latitude  <=  90) ? latitude  : 0
-
     readonly property real maxZoomLevel:    20
 
     zoomLevel:                  18
-    center:                     QtPositioning.coordinate(lat, lon)
+    center:                     QGroundControl.defaultMapPosition
     gesture.flickDeceleration:  3000
     gesture.enabled:            interactive
     gesture.activeGestures:     MapGestureArea.ZoomGesture | MapGestureArea.PanGesture | MapGestureArea.FlickGesture
@@ -69,6 +63,18 @@ Map {
 
     Component.onCompleted: onMapTypeChanged
 
+    property bool _initialMapPositionSet: false
+    Connections {
+        target: mainWindow
+
+        onGcsPositionChanged: {
+            if (!_initialMapPositionSet) {
+                _initialMapPositionSet = true
+                flightMap.center = mainWindow.gcsPosition
+            }
+        }
+    }
+
     onMapTypeChanged: {
         QGroundControl.flightMapSettings.setMapTypeForMapName(mapName, mapType)
         var fullMapName = QGroundControl.flightMapSettings.mapProvider + " " + mapType
@@ -80,7 +86,18 @@ Map {
         }
     }
 
-/*********************************************
+    MapQuickItem {
+        anchorPoint.x:  sourceItem.width  / 2
+        anchorPoint.y:  sourceItem.height / 2
+        visible:        mainWindow.gcsPosition.isValid
+        coordinate:     mainWindow.gcsPosition
+
+        sourceItem: MissionItemIndexLabel {
+            label: "Q"
+        }
+    }
+
+    /*********************************************
     /// Map control widgets
     Column {
         id:                 controlWidgets

src/MissionEditor/MissionEditor.qml

@@ -62,13 +62,8 @@ QGCView {
 
     property var    _missionItems:          controller.missionItems
     property var    _currentMissionItem
-
-    property bool   gpsLock:        _activeVehicle ? _activeVehicle.coordinateValid : false
-    property bool   _firstGpsLock:  true
-
-    //property var    _homePositionManager:       QGroundControl.homePositionManager
-    //property string _homePositionName:          _homePositionManager.homePositions.get(0).name
-    //property var    offlineHomePosition:        _homePositionManager.homePositions.get(0).coordinate
+    property bool   _firstVehiclePosition:  true
+    property var    activeVehiclePosition:  _activeVehicle ? _activeVehicle.coordinate : QtPositioning.coordinate()
 
     property var    liveHomePosition:           controller.liveHomePosition
     property var    liveHomePositionAvailable:  controller.liveHomePositionAvailable
@@ -76,17 +71,23 @@ QGCView {
 
     property bool _syncInProgress:              _activeVehicle ? _activeVehicle.missionManager.inProgress : false
 
-    onGpsLockChanged:       updateMapToVehiclePosition()
+    Component.onCompleted:          updateMapToVehiclePosition()
+    onActiveVehiclePositionChanged: updateMapToVehiclePosition()
 
-    Component.onCompleted: {
-        updateMapToVehiclePosition()
+    Connections {
+        target: multiVehicleManager
+
+        onActiveVehicleChanged: {
+            // When the active vehicle changes we need to allow the first vehicle position to move the map again
+            _firstVehiclePosition = true
+            updateMapToVehiclePosition()
+        }
     }
 
     function updateMapToVehiclePosition() {
-        if (gpsLock && _firstGpsLock) {
-            _firstGpsLock = false
-            editorMap.latitude = _activeVehicle.latitude
-            editorMap.longitude = _activeVehicle.longitude
+        if (_activeVehicle && _activeVehicle.coordinateValid && _firstVehiclePosition) {
+            _firstVehiclePosition = false
+            editorMap.center = _activeVehicle.coordinate
         }
     }
 
@@ -178,8 +179,6 @@ QGCView {
                 id:             editorMap
                 anchors.fill:   parent
                 mapName:        "MissionEditor"
-                latitude:       mainWindow.tabletPosition.latitude
-                longitude:      mainWindow.tabletPosition.longitude
 
                 readonly property real animationDuration: 500
 
@@ -190,20 +189,6 @@ QGCView {
                     }
                 }
 
-                Behavior on latitude {
-                    NumberAnimation {
-                        duration:       editorMap.animationDuration
-                        easing.type:    Easing.InOutQuad
-                    }
-                }
-
-                Behavior on longitude {
-                    NumberAnimation {
-                        duration:       editorMap.animationDuration
-                        easing.type:    Easing.InOutQuad
-                    }
-                }
-
                 MouseArea {
                     anchors.fill: parent
 

src/ui/MainWindowInner.qml

@@ -33,9 +33,9 @@ import QGroundControl.FlightDisplay         1.0
 import QGroundControl.ScreenTools           1.0
 import QGroundControl.MultiVehicleManager   1.0
 
-/// Inner common QML for MainWindow
+/// Inner common QML for mainWindow
 Item {
-    id:         mainWindow
+    id: mainWindow
 
     signal reallyClose
 
@@ -50,10 +50,7 @@ Item {
     property real   tbButtonWidth:      tbCellHeight * 1.35
     property real   availableHeight:    height - tbHeight
     property real   menuButtonWidth:    (tbButtonWidth * 2) + (tbSpacing * 4) + 1
-
-    property var    defaultPosition:    QtPositioning.coordinate(37.803784, -122.462276)
-    property var    tabletPosition:     defaultPosition
-
+    property var    gcsPosition:        QtPositioning.coordinate()  // Starts as invalid coordinate
     property var    currentPopUp:       null
     property real   currentCenterX:     0
     property var    activeVehicle:      multiVehicleManager.activeVehicle
@@ -179,20 +176,20 @@ Item {
     PositionSource {
         id:             positionSource
         updateInterval: 1000
-        active:         false
+        active:         true
+
         onPositionChanged: {
             if(positionSource.valid) {
                 if(positionSource.position.coordinate.latitude) {
                     if(Math.abs(positionSource.position.coordinate.latitude)  > 0.001) {
                         if(positionSource.position.coordinate.longitude) {
                             if(Math.abs(positionSource.position.coordinate.longitude)  > 0.001) {
-                                tabletPosition = positionSource.position.coordinate
+                                gcsPosition = positionSource.position.coordinate
                             }
                         }
                     }
                 }
             }
-            positionSource.stop()
         }
     }
 
@@ -299,9 +296,6 @@ Item {
         anchors.fill:       parent
         availableHeight:    mainWindow.availableHeight
         visible:            true
-        Component.onCompleted: {
-            positionSource.start()
-        }
     }
 
     Loader {