From de8988049d39d00c5325dd5d7e0d04f1e0d65a13 Mon Sep 17 00:00:00 2001
From: Uwe <donovaly@users.noreply.github.com>
Date: Sun, 26 Feb 2023 23:06:21 +0100
Subject: [PATCH] [App] [skip ci] some comment fixes

- also a sorting fix
---
 src/App/PropertyUnits.h           | 33 +++++++++++++------------------
 src/Gui/DlgUnitsCalculatorImp.cpp |  2 +-
 2 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/src/App/PropertyUnits.h b/src/App/PropertyUnits.h
index 4a70cf1fe0..b715d3e1af 100644
--- a/src/App/PropertyUnits.h
+++ b/src/App/PropertyUnits.h
@@ -246,9 +246,8 @@ public:
 };
 
 /** DynamicViscosity property
- * This is a property for representing dynamic viscosity. It is basically a
- * float property which must not be negative. On the Gui it has a quantity like
- * Pa*s.
+ * This is a property for representing dynamic viscosity. It is basically a float
+ * property which must not be negative. On the Gui it has a quantity like Pa*s.
  */
 class AppExport PropertyDynamicViscosity: public PropertyQuantity
 {
@@ -262,7 +261,7 @@ public:
 
 /** ElectricalCapacitance property
  * This is a property for representing capacitance. It is basically a float
- * property. On the Gui it has a quantity like uF.
+ * property. On the Gui it has a quantity like F.
  */
 class AppExport PropertyElectricalCapacitance: public PropertyQuantity
 {
@@ -413,9 +412,8 @@ public:
 };
 
 /** InverseArea property
- * This is a property for representing the reciprocal of area. It is basically a
- * float property which must not be negative. On the Gui it has a quantity like
- * 1/m^2.
+ * This is a property for representing the reciprocal of area. It is basically a float
+ * property which must not be negative. On the Gui it has a quantity like 1/m^2.
  */
 class AppExport PropertyInverseArea: public PropertyQuantity
 {
@@ -428,9 +426,8 @@ public:
 };
 
 /** InverseLength property
- * This is a property for representing the reciprocal of length. It is basically
- * a float property which must not be negative. On the Gui it has a quantity
- * like 1/m.
+ * This is a property for representing the reciprocal of length. It is basically a float
+ * property which must not be negative. On the Gui it has a quantity like 1/m.
  */
 class AppExport PropertyInverseLength: public PropertyQuantity
 {
@@ -443,9 +440,8 @@ public:
 };
 
 /** InverseVolume property
- * This is a property for representing the reciprocal of volume. It is basically
- * a float property. which must not be negative. On the Gui it has a quantity
- * like 1/m^3.
+ * This is a property for representing the reciprocal of volume. It is basically a float
+*  property. which must not be negative. On the Gui it has a quantity like 1/m^3.
  */
 class AppExport PropertyInverseVolume: public PropertyQuantity
 {
@@ -458,9 +454,8 @@ public:
 };
 
 /** KinematicViscosity property
- * This is a property for representing kinematic viscosity. It is basically a
- * float property which must not be negative. On the Gui it has a quantity like
- * m^2/s^2.
+ * This is a property for representing kinematic viscosity. It is basically a float
+ * property which must not be negative. On the Gui it has a quantity like m^2/s.
  */
 class AppExport PropertyKinematicViscosity: public PropertyQuantity
 {
@@ -501,7 +496,7 @@ public:
 
 /** MagneticFieldStrength property
  * This is a property for representing magnetic field strength. It is basically
- * a float property. On the Gui it has a quantity like Wb.
+ * a float property. On the Gui it has a quantity like A/m.
  */
 class AppExport PropertyMagneticFieldStrength: public PropertyQuantity
 {
@@ -611,7 +606,7 @@ public:
 
 /** SpecificEnergy property
  * This is a property for representing specific energy. It is basically a float
- * property. On the Gui it has a quantity like m^2/s^2.
+ * property. On the Gui it has a quantity like m^2/s^2 or J/kg.
  */
 class AppExport PropertySpecificEnergy: public PropertyQuantity
 {
@@ -665,7 +660,7 @@ public:
 
 /** Stress property
  * This is a property for representing . It is basically a float
- * property. On the Gui it has a quantity like .
+ * property. On the Gui it has a quantity like Pa.
  */
 class AppExport PropertyStress: public PropertyQuantity
 {
diff --git a/src/Gui/DlgUnitsCalculatorImp.cpp b/src/Gui/DlgUnitsCalculatorImp.cpp
index 9fe50e5b69..0b2e63fb6c 100644
--- a/src/Gui/DlgUnitsCalculatorImp.cpp
+++ b/src/Gui/DlgUnitsCalculatorImp.cpp
@@ -114,9 +114,9 @@ DlgUnitsCalculator::DlgUnitsCalculator( QWidget* parent, Qt::WindowFlags fl )
           << Base::Unit::TimeSpan
           << Base::Unit::VacuumPermittivity
           << Base::Unit::Velocity
+          << Base::Unit::Volume
           << Base::Unit::VolumeFlowRate
           << Base::Unit::VolumetricThermalExpansionCoefficient
-          << Base::Unit::Volume
           << Base::Unit::Work;
     for (QList<Base::Unit>::iterator it = units.begin(); it != units.end(); ++it) {
         ui->unitsBox->addItem(it->getTypeString());