Browse Source

Prevent message popup from scrolling map

Also remove concept of “interactive” since it is no longer needed
QGC4.4
Don Gagne 9 years ago
parent
commit
3eb9ab138b
  1. 6
      src/FlightDisplay/FlightDisplayView.qml
  2. 2
      src/FlightMap/FlightMap.qml
  3. 19
      src/ui/MainWindowInner.qml

6
src/FlightDisplay/FlightDisplayView.qml

@ -44,7 +44,6 @@ Item { @@ -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 { @@ -83,11 +82,6 @@ Item {
FlightDisplayViewController { id: _controller }
onInteractiveChanged: {
if(_flightMap)
_flightMap.interactive = interactive
}
function reloadContents() {
if(_flightVideo) {
_flightVideo.visible = false

2
src/FlightMap/FlightMap.qml

@ -43,7 +43,6 @@ import QGroundControl.Mavlink 1.0 @@ -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 { @@ -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" }

19
src/ui/MainWindowInner.qml

@ -201,7 +201,6 @@ Item { @@ -201,7 +201,6 @@ Item {
} else {
criticalMessageText.text = message
criticalMmessageArea.visible = true
mainWindow.setMapInteractive(false)
}
}
@ -221,10 +220,6 @@ Item { @@ -221,10 +220,6 @@ Item {
}
}
function setMapInteractive(enabled) {
flightView.interactive = enabled
}
onFormatedMessageChanged: {
if(messageArea.visible) {
messageText.append(formatedMessage)
@ -248,7 +243,6 @@ Item { @@ -248,7 +243,6 @@ Item {
}
currentPopUp = messageArea
messageArea.visible = true
mainWindow.setMapInteractive(false)
}
function showPopUp(dropItem, centerX) {
@ -341,7 +335,6 @@ Item { @@ -341,7 +335,6 @@ Item {
function close() {
currentPopUp = null
messageText.text = ""
mainWindow.setMapInteractive(true)
messageArea.visible = false
}
@ -405,7 +398,6 @@ Item { @@ -405,7 +398,6 @@ Item {
mainWindow.messageQueue = []
} else {
criticalMessageText.text = ""
mainWindow.setMapInteractive(true)
criticalMmessageArea.visible = false
}
}
@ -418,6 +410,15 @@ Item { @@ -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 { @@ -438,6 +439,7 @@ Item {
color: "#fdfd3b"
}
}
//-- Dismiss Critical Message
Image {
id: criticalClose
@ -457,6 +459,7 @@ Item { @@ -457,6 +459,7 @@ Item {
}
}
}
//-- More text below indicator
Image {
anchors.margins: ScreenTools.defaultFontPixelHeight

Loading…
Cancel
Save