feat(config): gate save/reset on body adjustment target selection

Disable the save and reset buttons until an adjustment target is selected,
since they have nothing to act on otherwise. Allow un-toggling the target
buttons so the selection can be cleared back to none, which now also resets
the active config target.

Author: ArthurHub

src/config-mode/BodyAdjustmentSubConfigMode.cpp

@@ -39,6 +39,11 @@ namespace frik
         _armsLengthMsg->setVisibility(_configTarget == BodyAdjustmentConfigTarget::BodyArmsLength);
         _vrScaleMsg->setVisibility(_configTarget == BodyAdjustmentConfigTarget::VRScale);
 
+        // save/reset only make sense once an adjustment target is selected
+        const bool noTarget = _configTarget == BodyAdjustmentConfigTarget::None;
+        _saveBtn->setDisabled(noTarget);
+        _resetBtn->setDisabled(noTarget);
+
         handleAdjustment();
     }
 
@@ -60,35 +65,45 @@ namespace frik
         row1Container->addElement(hideHeadBtn);
 
         const auto heightToggleBtn = std::make_shared<UIToggleButton>("ui-config-main\\btn-body-vertical.nif");
-        heightToggleBtn->setOnToggleHandler([this](UIWidget*, bool) { _configTarget = BodyAdjustmentConfigTarget::BodyHeight; });
+        heightToggleBtn->setOnToggleHandler(
+            [this](UIWidget*, const bool enabled) { _configTarget = enabled ? BodyAdjustmentConfigTarget::BodyHeight : BodyAdjustmentConfigTarget::None; });
 
         const auto forwardToggleBtn = std::make_shared<UIToggleButton>("ui-config-main\\btn-body-forward.nif");
-        forwardToggleBtn->setOnToggleHandler([this](UIWidget*, bool) { _configTarget = BodyAdjustmentConfigTarget::BodyForwardOffset; });
+        forwardToggleBtn->setOnToggleHandler(
+            [this](UIWidget*, const bool enabled) { _configTarget = enabled ? BodyAdjustmentConfigTarget::BodyForwardOffset : BodyAdjustmentConfigTarget::None; });
 
         const auto armsLengthToggleBtn = std::make_shared<UIToggleButton>("ui-config-main\\btn-arms-length.nif");
-        armsLengthToggleBtn->setOnToggleHandler([this](UIWidget*, bool) { _configTarget = BodyAdjustmentConfigTarget::BodyArmsLength; });
+        armsLengthToggleBtn->setOnToggleHandler(
+            [this](UIWidget*, const bool enabled) { _configTarget = enabled ? BodyAdjustmentConfigTarget::BodyArmsLength : BodyAdjustmentConfigTarget::None; });
 
         const auto vrScaleToggleBtn = std::make_shared<UIToggleButton>("ui-config-main\\btn-vr-scale.nif");
-        vrScaleToggleBtn->setOnToggleHandler([this](UIWidget*, bool) { _configTarget = BodyAdjustmentConfigTarget::VRScale; });
+        vrScaleToggleBtn->setOnToggleHandler(
+            [this](UIWidget*, const bool enabled) { _configTarget = enabled ? BodyAdjustmentConfigTarget::VRScale : BodyAdjustmentConfigTarget::None; });
 
         _row2Container = std::make_shared<UIToggleGroupContainer>("Row2", UIContainerLayout::HorizontalCenter, 0.3f);
         _row2Container->addElement(heightToggleBtn);
         _row2Container->addElement(forwardToggleBtn);
         _row2Container->addElement(armsLengthToggleBtn);
         _row2Container->addElement(vrScaleToggleBtn);
 
-        const auto saveBtn = std::make_shared<UIButton>("ui-common\\btn-save.nif");
-        saveBtn->setOnPressHandler([this](UIWidget*) { saveConfig(); });
+        // the toggle group disables un-toggling by default; re-enable it so the user can clear the selection back to no target
+        heightToggleBtn->setUnToggleAllowed(true);
+        forwardToggleBtn->setUnToggleAllowed(true);
+        armsLengthToggleBtn->setUnToggleAllowed(true);
+        vrScaleToggleBtn->setUnToggleAllowed(true);
+
+        _saveBtn = std::make_shared<UIButton>("ui-common\\btn-save.nif");
+        _saveBtn->setOnPressHandler([this](UIWidget*) { saveConfig(); });
 
-        const auto resetBtn = std::make_shared<UIButton>("ui-common\\btn-reset.nif");
-        resetBtn->setOnPressHandler([this](UIWidget*) { resetConfig(); });
+        _resetBtn = std::make_shared<UIButton>("ui-common\\btn-reset.nif");
+        _resetBtn->setOnPressHandler([this](UIWidget*) { resetConfig(); });
 
         const auto exitBtn = std::make_shared<UIButton>("ui-common\\btn-back.nif");
         exitBtn->setOnPressHandler([this](UIWidget*) { closeConfig(); });
 
         const auto row3Container = std::make_shared<UIContainer>("Row3", UIContainerLayout::HorizontalCenter, 0.3f);
-        row3Container->addElement(saveBtn);
-        row3Container->addElement(resetBtn);
+        row3Container->addElement(_saveBtn);
+        row3Container->addElement(_resetBtn);
         row3Container->addElement(exitBtn);
 
         _noneMsg = std::make_shared<UIWidget>("ui-config-main\\msg-node-selected.nif");

src/config-mode/BodyAdjustmentSubConfigMode.h

@@ -1,5 +1,6 @@
 #pragma once
 
+#include "vrui/UIButton.h"
 #include "vrui/UIContainer.h"
 #include "vrui/UIToggleGroupContainer.h"
 
@@ -53,6 +54,8 @@ namespace frik
         std::shared_ptr<vrui::UIContainer> _configUI;
 
         std::shared_ptr<vrui::UIToggleGroupContainer> _row2Container;
+        std::shared_ptr<vrui::UIButton> _saveBtn;
+        std::shared_ptr<vrui::UIButton> _resetBtn;
         std::shared_ptr<vrui::UIWidget> _noneMsg;
         std::shared_ptr<vrui::UIWidget> _heightMsg;
         std::shared_ptr<vrui::UIWidget> _forwardMsg;