Merge pull request #2624 from DonLakeFlyer/MessageMouseSteal

Prevent message popup from scrolling map

src/FlightDisplay/FlightDisplayView.qml

@@ -44,7 +44,6 @@ Item {
     QGCPalette { id: qgcPal; colorGroupEnabled: enabled }
 
     property real availableHeight: parent.height
-    property bool interactive:    true
 
     readonly property bool isBackgroundDark: _mainIsMap ? (_flightMap ? _flightMap.isSatelliteMap : true) : true
 
@@ -83,11 +82,6 @@ Item {
 
     FlightDisplayViewController { id: _controller }
 
-    onInteractiveChanged: {
-        if(_flightMap)
-            _flightMap.interactive = interactive
-    }
-
     function reloadContents() {
         if(_flightVideo) {
             _flightVideo.visible = false

src/FlightMap/FlightMap.qml

@@ -43,7 +43,6 @@ import QGroundControl.Mavlink               1.0
 Map {
     id: _map
 
-    property bool   interactive:        true
     property string mapName:            'defaultMap'
     property string mapType:            QGroundControl.flightMapSettings.mapTypeForMapName(mapName)
 //  property alias  mapWidgets:         controlWidgets
@@ -54,7 +53,6 @@ Map {
     zoomLevel:                  18
     center:                     QGroundControl.defaultMapPosition
     gesture.flickDeceleration:  3000
-    gesture.enabled:            interactive
     gesture.activeGestures:     MapGestureArea.ZoomGesture | MapGestureArea.PanGesture | MapGestureArea.FlickGesture
 
     plugin: Plugin { name: "QGroundControl" }

src/ui/MainWindowInner.qml

@@ -201,7 +201,6 @@ Item {
         } else {
             criticalMessageText.text = message
             criticalMmessageArea.visible = true
-            mainWindow.setMapInteractive(false)
         }
     }
 
@@ -221,10 +220,6 @@ Item {
         }
     }
 
-    function setMapInteractive(enabled) {
-        flightView.interactive = enabled
-    }
-
     onFormatedMessageChanged: {
         if(messageArea.visible) {
             messageText.append(formatedMessage)
@@ -248,7 +243,6 @@ Item {
         }
         currentPopUp = messageArea
         messageArea.visible = true
-        mainWindow.setMapInteractive(false)
     }
 
     function showPopUp(dropItem, centerX) {
@@ -341,7 +335,6 @@ Item {
         function close() {
             currentPopUp = null
             messageText.text    = ""
-            mainWindow.setMapInteractive(true)
             messageArea.visible = false
         }
 
@@ -405,7 +398,6 @@ Item {
                 mainWindow.messageQueue = []
             } else {
                 criticalMessageText.text = ""
-                mainWindow.setMapInteractive(true)
                 criticalMmessageArea.visible = false
             }
         }
@@ -418,6 +410,15 @@ Item {
         anchors.horizontalCenter:   parent.horizontalCenter
         anchors.bottom:             parent.bottom
         anchors.bottomMargin:       ScreenTools.defaultFontPixelHeight
+
+        MouseArea {
+            // This MouseArea prevents the Map below it from getting Mouse events. Without this
+            // things like mousewheel will scroll the Flickable and then scroll the map as well.
+            anchors.fill:       parent
+            preventStealing:    true
+            onWheel:            wheel.accepted = true
+        }
+
         Flickable {
             id:                 criticalMessageFlick
             anchors.margins:    ScreenTools.defaultFontPixelHeight
@@ -438,6 +439,7 @@ Item {
                 color:          "#fdfd3b"
             }
         }
+
         //-- Dismiss Critical Message
         Image {
             id:                 criticalClose
@@ -457,6 +459,7 @@ Item {
                 }
             }
         }
+
         //-- More text below indicator
         Image {
             anchors.margins:    ScreenTools.defaultFontPixelHeight