From 4fb46995774b0e4a0026370903601aa992b638e5 Mon Sep 17 00:00:00 2001
From: DonLakeFlyer <dongagne@outlook.com>
Date: Wed, 25 Dec 2019 08:49:18 -0800
Subject: [PATCH] Persist MissisonItemStatus visibility

---
 src/PlanView/PlanView.qml                | 4 +++-
 src/Settings/PlanView.SettingsGroup.json | 6 ++++++
 src/Settings/PlanViewSettings.cc         | 1 +
 src/Settings/PlanViewSettings.h          | 1 +
 4 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/PlanView/PlanView.qml b/src/PlanView/PlanView.qml
index ca988cb38..84c0ce63e 100644
--- a/src/PlanView/PlanView.qml
+++ b/src/PlanView/PlanView.qml
@@ -52,6 +52,7 @@ Item {
     property int    _editingLayer:                      bar.currentIndex ? _layers[bar.currentIndex] : _layerMission
     property int    _toolStripBottom:                   toolStrip.height + toolStrip.y
     property var    _appSettings:                       QGroundControl.settingsManager.appSettings
+    property var    _planViewSettings:                  QGroundControl.settingsManager.planViewSettings
 
     readonly property var       _layers:                [_layerMission, _layerGeoFence, _layerRallyPoints]
 
@@ -886,10 +887,11 @@ Item {
 
             onSetCurrentSeqNum: _missionController.setCurrentPlanViewSeqNum(seqNum, true)
 
-            property bool _internalVisible: false
+            property bool _internalVisible: _planViewSettings.showMissionItemStatus.rawValue
 
             function toggleVisible() {
                 _internalVisible = !_internalVisible
+                _planViewSettings.showMissionItemStatus.rawValue = _internalVisible
             }
         }
 
diff --git a/src/Settings/PlanView.SettingsGroup.json b/src/Settings/PlanView.SettingsGroup.json
index c87293be0..eb1921d9d 100644
--- a/src/Settings/PlanView.SettingsGroup.json
+++ b/src/Settings/PlanView.SettingsGroup.json
@@ -10,5 +10,11 @@
     "shortDescription": "Don't warn user about 'Above Terrain' usage",
     "type":             "bool",
     "defaultValue":     false
+},
+{
+    "name":             "showMissionItemStatus",
+    "shortDescription": "Show/Hide the mission item status display",
+    "type":             "bool",
+    "defaultValue":     false
 }
 ]
diff --git a/src/Settings/PlanViewSettings.cc b/src/Settings/PlanViewSettings.cc
index 9c3e88da9..3bd9c2de6 100644
--- a/src/Settings/PlanViewSettings.cc
+++ b/src/Settings/PlanViewSettings.cc
@@ -19,3 +19,4 @@ DECLARE_SETTINGGROUP(PlanView, "PlanView")
 
 DECLARE_SETTINGSFACT(PlanViewSettings, displayPresetsTabFirst)
 DECLARE_SETTINGSFACT(PlanViewSettings, aboveTerrainWarning)
+DECLARE_SETTINGSFACT(PlanViewSettings, showMissionItemStatus)
diff --git a/src/Settings/PlanViewSettings.h b/src/Settings/PlanViewSettings.h
index 1c8e670ed..3ea708b45 100644
--- a/src/Settings/PlanViewSettings.h
+++ b/src/Settings/PlanViewSettings.h
@@ -22,4 +22,5 @@ public:
 
     DEFINE_SETTINGFACT(displayPresetsTabFirst)
     DEFINE_SETTINGFACT(aboveTerrainWarning)
+    DEFINE_SETTINGFACT(showMissionItemStatus)
 };
-- 
2.22.0