From c9f273a47aa5bf071ad74f5f01260a7d38eaec4b Mon Sep 17 00:00:00 2001
From: Hari Rana <theevilskeleton@riseup.net>
Date: Thu, 30 Apr 2026 21:23:17 -0400
Subject: [PATCH] event-editor/multi-choice: Consolidate action activation

Part-of: <https://gitlab.gnome.org/GNOME/gnome-calendar/-/merge_requests/754>
---
 src/gui/event-editor/gcal-multi-choice.c | 30 +++++++++++++-----------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/src/gui/event-editor/gcal-multi-choice.c b/src/gui/event-editor/gcal-multi-choice.c
index 12221ade..7978d96b 100644
--- a/src/gui/event-editor/gcal-multi-choice.c
+++ b/src/gui/event-editor/gcal-multi-choice.c
@@ -155,12 +155,10 @@ set_value (GcalMultiChoice         *self,
 }
 
 static void
-up_action_activated (GcalMultiChoice *self)
+action_activated (GcalMultiChoice *self,
+                  int              value)
 {
-  int value, wrapped_value;
-
-  value = self->next_cb ? self->next_cb (self->value) : self->value + 1;
-  g_assert_cmpint (value, >, self->value);
+  int wrapped_value;
 
   wrapped_value = WRAP (value, self->min_value, self->max_value);
 
@@ -172,22 +170,26 @@ up_action_activated (GcalMultiChoice *self)
   gtk_widget_grab_focus (GTK_WIDGET (self));
 }
 
+static void
+up_action_activated (GcalMultiChoice *self)
+{
+  int value;
+
+  value = self->next_cb ? self->next_cb (self->value) : self->value + 1;
+  g_assert_cmpint (value, >, self->value);
+
+  action_activated (self, value);
+}
+
 static void
 down_action_activated (GcalMultiChoice *self)
 {
-  int value, wrapped_value;
+  int value;
 
   value = self->prev_cb ? self->prev_cb (self->value) : self->value - 1;
   g_assert_cmpint (value, <, self->value);
 
-  wrapped_value = WRAP (value, self->min_value, self->max_value);
-
-  set_value (self, wrapped_value, GTK_STACK_TRANSITION_TYPE_NONE);
-
-  if (wrapped_value != value)
-    g_signal_emit (self, signals[WRAPPED], 0);
-
-  gtk_widget_grab_focus (GTK_WIDGET (self));
+  action_activated (self, value);
 }
 
 static void