Merge pull request #3441 from DonLakeFlyer/SummaryUnits

Show units in summary items

src/AutoPilotPlugins/APM/APMPowerComponentSummary.qml

@@ -50,7 +50,7 @@ FactPanel {
 
         VehicleSummaryRow {
             labelText: qsTr("Battery capacity:")
-            valueText: battCapacity.valueString
+            valueText: battCapacity.valueString + " " + battCapacity.units
         }
     }
 }

src/AutoPilotPlugins/APM/APMSafetyComponentSummaryCopter.qml

@@ -120,22 +120,22 @@ FactPanel {
 
         VehicleSummaryRow {
             labelText: qsTr("RTL min alt:")
-            valueText: _rtlAltFact.value == 0 ? qsTr("current") : _rtlAltFact.valueString
+            valueText: _rtlAltFact.value == 0 ? qsTr("current") : _rtlAltFact.valueString + " " + _rtlAltFact.units
         }
 
         VehicleSummaryRow {
             labelText: qsTr("RTL loiter time:")
-            valueText: _rtlLoitTimeFact.valueString
+            valueText: _rtlLoitTimeFact.valueString + " " + _rtlLoitTimeFact.units
         }
 
         VehicleSummaryRow {
             labelText: qsTr("RTL final alt:")
-            valueText: _rtlAltFinalFact.value == 0 ? qsTr("Land") : _rtlAltFinalFact.valueString
+            valueText: _rtlAltFinalFact.value == 0 ? qsTr("Land") : _rtlAltFinalFact.valueString + " " + _rtlAltFinalFact.units
         }
 
         VehicleSummaryRow {
             labelText: qsTr("Descent speed:")
-            valueText: _landSpeedFact.valueString
+            valueText: _landSpeedFact.valueString + " " + _landSpeedFact.units
         }
     }
 }

src/AutoPilotPlugins/APM/APMSafetyComponentSummaryPlane.qml

@@ -32,17 +32,17 @@ FactPanel {
 
         VehicleSummaryRow {
             labelText: qsTr("Voltage failsafe:")
-            valueText:  _failsafeBattVoltage.value == 0 ? qsTr("Disabled") : _failsafeBattVoltage.valueString
+            valueText:  _failsafeBattVoltage.value == 0 ? qsTr("Disabled") : _failsafeBattVoltage.valueString + " " + _failsafeBattVoltage.units
         }
 
         VehicleSummaryRow {
             labelText: qsTr("mAh failsafe:")
-            valueText:  _failsafeBattMah.value == 0 ? qsTr("Disabled") : _failsafeBattMah.valueString
+            valueText:  _failsafeBattMah.value == 0 ? qsTr("Disabled") : _failsafeBattMah.valueString + " " + _failsafeBattMah.units
         }
 
         VehicleSummaryRow {
             labelText: qsTr("RTL min alt:")
-            valueText: _rtlAltFact.value < 0 ? qsTr("current") : _rtlAltFact.valueString
+            valueText: _rtlAltFact.value < 0 ? qsTr("current") : _rtlAltFact.valueString + " " + _rtlAltFact.units
         }
     }
 }

src/AutoPilotPlugins/PX4/PowerComponentSummary.qml

@@ -50,12 +50,12 @@ FactPanel {
 
         VehicleSummaryRow {
             labelText: qsTr("Battery Full:")
-            valueText: batVChargedFact ? batVChargedFact.valueString : ""
+            valueText: batVChargedFact ? batVChargedFact.valueString + " " + batVChargedFact.units : ""
         }
 
         VehicleSummaryRow {
             labelText: qsTr("Battery Empty:")
-            valueText: batVEmptyFact ? batVEmptyFact.valueString : ""
+            valueText: batVEmptyFact ? batVEmptyFact.valueString + " " + batVEmptyFact.units : ""
         }
 
         VehicleSummaryRow {

src/AutoPilotPlugins/PX4/SafetyComponentSummary.qml

@@ -27,17 +27,17 @@ FactPanel {
 
         VehicleSummaryRow {
             labelText: qsTr("RTL min alt:")
-            valueText: returnAltFact ? returnAltFact.valueString : ""
+            valueText: returnAltFact ? returnAltFact.valueString + " " + returnAltFact.units : ""
         }
 
         VehicleSummaryRow {
             labelText: qsTr("RTL home alt:")
-            valueText: descendAltFact ? descendAltFact.valueString : ""
+            valueText: descendAltFact ? descendAltFact.valueString + " " + descendAltFact.units : ""
         }
 
         VehicleSummaryRow {
-            labelText: qsTr("RC loss RTL (seconds):")
-            valueText: commRCLossFact ? commRCLossFact.valueString : ""
+            labelText: qsTr("RC loss RTL:")
+            valueText: commRCLossFact ? commRCLossFact.valueString + " " + commRCLossFact.units : ""
         }
 
         VehicleSummaryRow {