Merge pull request #4645 from DonLakeFlyer/IndoorPalette

Fix indoorPalette overriding

src/Settings/AppSettings.cc

@@ -47,15 +47,7 @@ AppSettings::AppSettings(QObject* parent)
 {
     QQmlEngine::setObjectOwnership(this, QQmlEngine::CppOwnership);
     qmlRegisterUncreatableType<AppSettings>("QGroundControl.SettingsManager", 1, 0, "AppSettings", "Reference only");        
-
-    // Set up correct default for palette setting
-    QVariant outdoorPalette;
-#if defined (__mobile__)
-    outdoorPalette = 0;
-#else
-    outdoorPalette = 1;
-#endif
-    _nameToMetaDataMap[indoorPaletteName]->setRawDefaultValue(outdoorPalette);
+    QGCPalette::setGlobalTheme(indoorPalette()->rawValue().toBool() ? QGCPalette::Dark : QGCPalette::Light);
 }
 
 Fact* AppSettings::offlineEditingFirmwareType(void)
@@ -177,5 +169,5 @@ Fact* AppSettings::indoorPalette(void)
 void AppSettings::_indoorPaletteChanged(void)
 {
     qgcApp()->_loadCurrentStyleSheet();
-    QGCPalette::setGlobalTheme(_indoorPaletteFact->rawValue().toBool() ? QGCPalette::Dark : QGCPalette::Light);
+    QGCPalette::setGlobalTheme(indoorPalette()->rawValue().toBool() ? QGCPalette::Dark : QGCPalette::Light);
 }

src/api/QGCCorePlugin.cc

@@ -11,6 +11,7 @@
 #include "QGCOptions.h"
 #include "QGCSettings.h"
 #include "FactMetaData.h"
+#include "SettingsManager.h"
 
 #include <QtQml>
 #include <QQmlEngine>
@@ -164,6 +165,16 @@ bool QGCCorePlugin::overrideSettingsGroupVisibility(QString name)
 
 bool QGCCorePlugin::adjustSettingMetaData(FactMetaData& metaData)
 {
-    Q_UNUSED(metaData); // No mods to standard meta data
+    if (metaData.name() == AppSettings::indoorPaletteName) {
+        // Set up correct default for palette setting
+        QVariant outdoorPalette;
+#if defined (__mobile__)
+        outdoorPalette = 0;
+#else
+        outdoorPalette = 1;
+#endif
+        metaData.setRawDefaultValue(outdoorPalette);
+    }
+
     return true;        // Show setting in ui
 }