Skip to content
Permalink
Browse files

Drop eom_close_confirmation_dialog_set_sensitive

This internal function is only used at one place.
No need to set the dialog buttons insensitive after
save-image is executed.

As result we don't use deprecated gtk_dialog_get_action_area

inspired by comment from
https://gitlab.gnome.org/GNOME/eog/commit/a48af02
  • Loading branch information...
raveit65 committed Jul 22, 2018
1 parent 704107b commit 9593351e473942268e6297011634d75246d4e246
Showing with 0 additions and 18 deletions.
  1. +0 −14 src/eom-close-confirmation-dialog.c
  2. +0 −3 src/eom-close-confirmation-dialog.h
  3. +0 −1 src/eom-window.c
@@ -676,17 +676,3 @@ eom_close_confirmation_dialog_get_unsaved_images (EomCloseConfirmationDialog *dl
return dlg->priv->unsaved_images;
}

void
eom_close_confirmation_dialog_set_sensitive (EomCloseConfirmationDialog *dlg,
gboolean value)
{
GtkWidget *action_area;

g_return_if_fail (EOM_IS_CLOSE_CONFIRMATION_DIALOG (dlg));

action_area = gtk_dialog_get_action_area (GTK_DIALOG (dlg));
gtk_widget_set_sensitive (action_area, value);

if (dlg->priv->toggle_renderer)
gtk_cell_renderer_toggle_set_activatable (GTK_CELL_RENDERER_TOGGLE (dlg->priv->toggle_renderer), value);
}
@@ -72,8 +72,5 @@ const GList *eom_close_confirmation_dialog_get_unsaved_images (EomCloseConfirmat
G_GNUC_INTERNAL
GList *eom_close_confirmation_dialog_get_selected_images (EomCloseConfirmationDialog *dlg);

G_GNUC_INTERNAL
void eom_close_confirmation_dialog_set_sensitive (EomCloseConfirmationDialog *dlg, gboolean value);

#endif /* __EOM_CLOSE_CONFIRMATION_DIALOG_H__ */

@@ -2300,7 +2300,6 @@ close_confirmation_dialog_response_handler (EomCloseConfirmationDialog *dlg,
case GTK_RESPONSE_YES:
/* save selected images */
selected_images = eom_close_confirmation_dialog_get_selected_images (dlg);
eom_close_confirmation_dialog_set_sensitive (dlg, FALSE);
if (eom_window_save_images (window, selected_images)) {
g_signal_connect (priv->save_job,
"finished",

0 comments on commit 9593351

Please sign in to comment.
You can’t perform that action at this time.