Refactor WidgetRange increment and decrement methods

The code now simplifies the increment and decrement methods in the WidgetRange class, improving readability and maintainability.
forntoh · Oct 22, 2024 · d2f055e · d2f055e
1 parent 548996b
commit d2f055e
Showing 1 changed file with 9 additions and 10 deletions.
diff --git a/src/widget/WidgetRange.h b/src/widget/WidgetRange.h
@@ -4,6 +4,11 @@
 
 #include "BaseWidgetValue.h"
 
+/**
+ * @class WidgetRange
+ * @brief Widget that allows user to select a value from a range.
+ * Manages a value within a specified range, allowing incrementing and decrementing.
+ */
 template <typename T>
 class WidgetRange : public BaseWidgetValue<T> {
   protected:
@@ -49,17 +54,11 @@ class WidgetRange : public BaseWidgetValue<T> {
         if (renderer->isInEditMode()) {
             switch (command) {
                 case UP:
-                    if (increment()) {
-                        BaseWidgetValue<T>::handleChange();
-                        return true;
-                    }
-                    return false;
+                    if (increment()) BaseWidgetValue<T>::handleChange();
+                    return true;
                 case DOWN:
-                    if (decrement()) {
-                        BaseWidgetValue<T>::handleChange();
-                        return true;
-                    }
-                    return false;
+                    if (decrement()) BaseWidgetValue<T>::handleChange();
+                    return true;
                 default:
                     return false;
             }