Federated_Computer/federated-calendar
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Start transitioning gcal_schedule_section_set_event() to use update_widgets()

Browse Source 
For now this is redundant, as update_widgets() only sets the
visibility of the time selectors, while the call to
adw_toggle_group_set_active_name() effectively does the same.

But that redundancy will go away in the next step.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-calendar/-/merge_requests/550>
This commit is contained in:
Federico Mena Quintero
2025-08-27 19:59:17 -03:00
committed by Georges Basile Stavracas Neto
parent 640488883b
commit 253b1d4f17
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/gui/event-editor/gcal-schedule-section.c
+4 -4
View File
@@ -188,6 +188,7 @@ static void
update_widgets (GcalScheduleSection *self,
WidgetState *state)
{
gtk_widget_set_visible (GTK_WIDGET (self->start_date_group), state->date_widgets_visible);
gtk_widget_set_visible (GTK_WIDGET (self->end_date_group), state->date_widgets_visible);
gtk_widget_set_visible (GTK_WIDGET (self->start_date_time_chooser), state->date_time_widgets_visible);
@@ -410,6 +411,7 @@ gcal_schedule_section_set_event (GcalEventEditorSection *section,
GcalRecurrenceFrequency frequency;
gboolean all_day, new_event;
GcalScheduleValues *values;
g_autoptr (WidgetState) state = NULL;
GCAL_ENTRY;
@@ -431,10 +433,8 @@ gcal_schedule_section_set_event (GcalEventEditorSection *section,
adw_toggle_group_set_active_name (self->schedule_type_toggle_group,
all_day ? "all-day" : "time-slot");
gtk_widget_set_visible (GTK_WIDGET (self->start_date_group), all_day);
gtk_widget_set_visible (GTK_WIDGET (self->end_date_group), all_day);
gtk_widget_set_visible (GTK_WIDGET (self->start_date_time_chooser), !all_day);
gtk_widget_set_visible (GTK_WIDGET (self->end_date_time_chooser), !all_day);
state = widget_state_from_values (values);
update_widgets (self, state);
/* retrieve start and end date-times */
date_time_start = values->date_start;