From d59db82eadf2f2e718f0de79698656f815a72d03 Mon Sep 17 00:00:00 2001
From: "Ben V. Brown" <5425387+Ralim@users.noreply.github.com>
Date: Sat, 15 Jan 2022 08:52:27 +1100
Subject: [PATCH] Update Settings.cpp

---
 source/Core/Src/Settings.cpp | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/source/Core/Src/Settings.cpp b/source/Core/Src/Settings.cpp
index e09501c3..7d074da6 100644
--- a/source/Core/Src/Settings.cpp
+++ b/source/Core/Src/Settings.cpp
@@ -172,13 +172,18 @@ bool nextSettingValue(const enum SettingsOptions option) {
 // Return true if we are at the end (min)
 bool prevSettingValue(const enum SettingsOptions option) {
   const auto constants = settingsConstants[(int)option];
-  int        value     = systemSettings.settingsValues[(int)option];
-  if (value <= constants.min) {
-    value = constants.max;
+  if (systemSettings.settingsValues[(int)option] == (constants.min)) {
+    // Already at min, wrap to the max
+    systemSettings.settingsValues[(int)option] = constants.max;
+  } else if (systemSettings.settingsValues[(int)option] <= (constants.min + constants.increment)) {
+    // If within one increment of the start, constrain to the start
+    systemSettings.settingsValues[(int)option] = constants.min;
+  } else {
+    // Otherwise decrement
+    systemSettings.settingsValues[(int)option] -= constants.increment;
   }
-  value -= constants.increment;
-  systemSettings.settingsValues[(int)option] = value;
-  return systemSettings.settingsValues[(int)option] == constants.min;
+  // Return if we are at the min
+  return constants.min == systemSettings.settingsValues[(int)option];
 }
 uint16_t lookupHallEffectThreshold() {
   // Return the threshold above which the hall effect sensor is "activated"