Improve marker mag threshold setting

Stellarium · Aug 26, 2024 · 16b08f6 · 16b08f6
1 parent 4134638
commit 16b08f6
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/src/core/modules/SolarSystem.cpp b/src/core/modules/SolarSystem.cpp
@@ -3929,3 +3929,10 @@ void SolarSystem::setExtraThreads(int n)
 	StelApp::immediateSave("astro/solar_system_threads", extraThreads);
 	emit extraThreadsChanged(extraThreads);
 }
+
+void SolarSystem::setMarkerMagThreshold(double m)
+{
+	markerMagThreshold=qBound(-2.,m,37.);
+	StelApp::immediateSave("astro/planet_markers_mag_threshold", markerMagThreshold);
+	emit markerMagThresholdChanged(markerMagThreshold);
+}
diff --git a/src/core/modules/SolarSystem.hpp b/src/core/modules/SolarSystem.hpp
@@ -755,10 +755,10 @@ public slots:
 	//! The argument will be bounded by 0 and QThreadPool::globalInstance()->maxThreadCount()-1
 	void setExtraThreads(int n);
 
-	//! Return the limiting absolute magnitude configured for plotting minor bodies. 30 means "all".
+	//! Return the limiting absolute magnitude configured for plotting minor bodies. (min. mag. 37 includes de facto "all".)
 	double getMarkerMagThreshold() const {return markerMagThreshold;}
-	//! Configure the limiting absolute magnitude for plotting minor bodies. Configured value is clamped to -2..30, 30 means "all".
-	void setMarkerMagThreshold(double m) {markerMagThreshold=qBound(-2.,m,35.); emit markerMagThresholdChanged(markerMagThreshold);}
+	//! Configure the limiting absolute magnitude for plotting minor bodies. Configured value is clamped to -2..37 (practical limit)
+	void setMarkerMagThreshold(double m);
 
 signals:
 	void labelsDisplayedChanged(bool b);

diff --git a/src/gui/viewDialog.ui b/src/gui/viewDialog.ui
@@ -1425,12 +1425,15 @@
                    <property name="toolTip">
                     <string>Draw markers for objects with magnitude brighter than this</string>
                    </property>
-                   <property name="maximum">
-                    <double>35.000000000000000</double>
+                   <property name="decimals">
+                    <number>1</number>
                    </property>
                    <property name="minimum">
                     <double>5.000000000000000</double>
                    </property>
+                   <property name="maximum">
+                    <double>37.000000000000000</double>
+                   </property>
                    <property name="singleStep">
                     <double>0.100000000000000</double>
                    </property>