From 5aea83e725be4c382656e6364632ea6c31c8f679 Mon Sep 17 00:00:00 2001
From: Scott Moreau <oreaus@gmail.com>
Date: Sat, 29 Jun 2024 09:23:39 -0600
Subject: [PATCH] Fix reset of int options on OptionWidget destructor (#85)

Int durations were set to 0 after adjusting an int or animation option
and susequently clicking the back button or closing wcm. Fix this by
using the solution found in gtkmm bug #128 on gitlab. Thanks to ammen99
for pointing out the initial solution.
---
 src/wcm.cpp | 10 +++++-----
 src/wcm.hpp |  1 +
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/wcm.cpp b/src/wcm.cpp
index 2315fbc..e20b135 100644
--- a/src/wcm.cpp
+++ b/src/wcm.cpp
@@ -347,11 +347,11 @@ OptionWidget::OptionWidget(Option *option) : Gtk::Box(Gtk::ORIENTATION_HORIZONTA
             auto spin_button = std::make_unique<Gtk::SpinButton>(
                 Gtk::Adjustment::create(value, option->data.min, option->data.max,
                     1));
-            spin_button->signal_value_changed().connect(
+            spin_button->signal_changed().connect(sigc::track_obj(
                 [=, widget = spin_button.get()]
                 {
                     option->set_save(widget->get_value_as_int());
-                });
+                }, tracker));
             reset_button.signal_clicked().connect(
                 [=, widget = spin_button.get()]
                 {
@@ -423,7 +423,7 @@ OptionWidget::OptionWidget(Option *option) : Gtk::Box(Gtk::ORIENTATION_HORIZONTA
                 option->set_save(std::to_string(
                     length_widget->get_value_as_int()) + "ms " + easing_widget->get_active_text().c_str());
             };
-        spin_button->signal_changed().connect(update_option_value);
+        spin_button->signal_changed().connect(sigc::track_obj(update_option_value, tracker));
         combo_box->signal_changed().connect(std::move(update_option_value));
         reset_button.signal_clicked().connect([=, length_widget = spin_button.get(),
                                                easing_widget = combo_box.get()]
@@ -470,10 +470,10 @@ OptionWidget::OptionWidget(Option *option) : Gtk::Box(Gtk::ORIENTATION_HORIZONTA
             Gtk::Adjustment::create(value, option->data.min, option->data.max,
                 option->data.precision),
             option->data.precision, 3);
-        spin_box->signal_changed().connect([=, widget = spin_box.get()]
+        spin_box->signal_changed().connect(sigc::track_obj([=, widget = spin_box.get()]
             {
                 option->set_save(widget->get_value());
-            });
+            }, tracker));
         reset_button.signal_clicked().connect(
             [=, widget = spin_box.get()]
             {
diff --git a/src/wcm.hpp b/src/wcm.hpp
index 8324424..1aad888 100644
--- a/src/wcm.hpp
+++ b/src/wcm.hpp
@@ -138,6 +138,7 @@ class OptionWidget : public Gtk::Box
     Gtk::Label name_label;
     std::vector<std::unique_ptr<Gtk::Widget>> widgets;
     Gtk::Button reset_button;
+    sigc::trackable tracker;
 
     inline void pack_end(std::unique_ptr<Gtk::Widget> && widget, bool expand = false,
         bool fill = false)