From 4f037132febbb48e7ecea0176c92e087ab2afd00 Mon Sep 17 00:00:00 2001
From: Hari Rana <theevilskeleton@riseup.net>
Date: Sun, 1 Mar 2026 09:20:01 -0500
Subject: [PATCH] event-editor/schedule-section: Use `separate-rows` property
 in all-day section

`separate-rows` already does what we want, and using that property
allows us to nesting AdwPreferencesGroup into a GtkBox widget.

Co-authored-by: Jamie Gravendeel <me@jamie.garden>
Part-of: <https://gitlab.gnome.org/GNOME/gnome-calendar/-/merge_requests/781>
---
 .../event-editor/gcal-schedule-section.blp    | 21 +++++++------------
 src/gui/event-editor/gcal-schedule-section.c  |  4 ----
 2 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/src/gui/event-editor/gcal-schedule-section.blp b/src/gui/event-editor/gcal-schedule-section.blp
index 262340a6..3e22175a 100644
--- a/src/gui/event-editor/gcal-schedule-section.blp
+++ b/src/gui/event-editor/gcal-schedule-section.blp
@@ -18,22 +18,17 @@ template $GcalScheduleSection: Box {
       name: "all-day";
       use-underline: true;
 
-      child: Box {
-        orientation: vertical;
-        spacing: 12;
+      child: Adw.PreferencesGroup {
+        separate-rows: true;
 
-        Adw.PreferencesGroup start_date_group {
-          $GcalDateChooserRow start_date_row {
-            title: _("Start Date");
-            notify::date => $on_start_date_changed_cb(template) not-swapped;
-          }
+        $GcalDateChooserRow start_date_row {
+          title: _("Start Date");
+          notify::date => $on_start_date_changed_cb(template) not-swapped;
         }
 
-        Adw.PreferencesGroup end_date_group {
-          $GcalDateChooserRow end_date_row {
-            title: _("End Date");
-            notify::date => $on_end_date_changed_cb(template) not-swapped;
-          }
+        $GcalDateChooserRow end_date_row {
+          title: _("End Date");
+          notify::date => $on_end_date_changed_cb(template) not-swapped;
         }
       };
     }
diff --git a/src/gui/event-editor/gcal-schedule-section.c b/src/gui/event-editor/gcal-schedule-section.c
index b9962e6d..861557d2 100644
--- a/src/gui/event-editor/gcal-schedule-section.c
+++ b/src/gui/event-editor/gcal-schedule-section.c
@@ -41,9 +41,7 @@ struct _GcalScheduleSection
   GcalEventSchedule *values;
 
   AdwViewStack        *schedule_type_stack;
-  AdwPreferencesGroup *start_date_group;
   GcalDateChooserRow  *start_date_row;
-  AdwPreferencesGroup *end_date_group;
   GcalDateChooserRow  *end_date_row;
   GcalDateTimeChooser *start_date_time_chooser;
   GcalDateTimeChooser *end_date_time_chooser;
@@ -656,9 +654,7 @@ gcal_schedule_section_class_init (GcalScheduleSectionClass *klass)
   gtk_widget_class_set_template_from_resource (widget_class, "/org/gnome/calendar/ui/event-editor/gcal-schedule-section.ui");
 
   gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, schedule_type_stack);
-  gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, start_date_group);
   gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, start_date_row);
-  gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, end_date_group);
   gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, end_date_row);
   gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, start_date_time_chooser);
   gtk_widget_class_bind_template_child (widget_class, GcalScheduleSection, end_date_time_chooser);