From 9d9d538fe6c55d6dc792a75fe7d0e2d8f7af415c Mon Sep 17 00:00:00 2001 From: Přemysl Eric Janouch Date: Tue, 30 May 2023 10:19:27 +0200 Subject: Add trashing to file context menus --- fiv-context-menu.c | 38 ++++++++++++++++++++++++++++++++++++-- 1 file changed, 36 insertions(+), 2 deletions(-) diff --git a/fiv-context-menu.c b/fiv-context-menu.c index 3ee211d..d9dae62 100644 --- a/fiv-context-menu.c +++ b/fiv-context-menu.c @@ -327,6 +327,21 @@ open_context_unref(gpointer data, G_GNUC_UNUSED GClosure *closure) g_rc_box_release_full(data, open_context_finalize); } +static void +open_context_show_error_dialog(OpenContext *self, GError *error) +{ + GtkWindow *window = g_weak_ref_get(&self->window); + + GtkWidget *dialog = + gtk_message_dialog_new(GTK_WINDOW(window), GTK_DIALOG_MODAL, + GTK_MESSAGE_ERROR, GTK_BUTTONS_CLOSE, "%s", error->message); + gtk_dialog_run(GTK_DIALOG(dialog)); + gtk_widget_destroy(dialog); + + g_clear_object(&window); + g_error_free(error); +} + static void open_context_launch(GtkWidget *widget, OpenContext *self) { @@ -342,8 +357,7 @@ open_context_launch(GtkWidget *widget, OpenContext *self) (void) g_app_info_set_as_last_used_for_type( self->app_info, self->content_type, NULL); } else { - g_warning("%s", error->message); - g_error_free(error); + open_context_show_error_dialog(self, error); } g_list_free(files); g_object_unref(context); @@ -423,6 +437,16 @@ on_info_activate(G_GNUC_UNUSED GtkMenuItem *item, gpointer user_data) g_free(uri); } +static void +on_trash_activate(G_GNUC_UNUSED GtkMenuItem *item, gpointer user_data) +{ + // TODO(p): Use g_file_trash_async(), for which we need a task manager. + OpenContext *ctx = user_data; + GError *error = NULL; + if (!g_file_trash(ctx->file, NULL, &error)) + open_context_show_error_dialog(ctx, error); +} + static gboolean destroy_widget_idle_source_func(GtkWidget *widget) { @@ -503,6 +527,16 @@ fiv_context_menu_new(GtkWidget *widget, GFile *file) ctx, open_context_unref, 0); gtk_menu_shell_append(GTK_MENU_SHELL(menu), item); + // TODO(p): Can we avoid using the "trash" string constant for this check? + if (!g_file_has_uri_scheme(file, "trash")) { + gtk_menu_shell_append( + GTK_MENU_SHELL(menu), gtk_separator_menu_item_new()); + + item = gtk_menu_item_new_with_mnemonic("Move to _Trash"); + g_signal_connect_data(item, "activate", G_CALLBACK(on_trash_activate), + g_rc_box_acquire(ctx), open_context_unref, 0); + gtk_menu_shell_append(GTK_MENU_SHELL(menu), item); + } if (type == G_FILE_TYPE_REGULAR) { gtk_menu_shell_append( GTK_MENU_SHELL(menu), gtk_separator_menu_item_new()); -- cgit v1.2.3-70-g09d2