Merged in CMaiku/pidgin (pull request #372)

Wed, 13 Jun 2018 03:18:40 +0000

author
Gary Kramlich <grim@reaperworld.com>
date
Wed, 13 Jun 2018 03:18:40 +0000
changeset 39151
4d28b6e4f277
parent 39149
c18e9cfa0310 (current diff)
parent 39150
dcbc4df3018f (diff)
child 39164
d4573bd02c98

Merged in CMaiku/pidgin (pull request #372)

gtkxfer: Replace gtk_widget_render_icon() with icon names

Approved-by: Gary Kramlich <grim@reaperworld.com>
Approved-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>

--- a/pidgin/gtkxfer.c	Wed Jun 13 03:16:37 2018 +0000
+++ b/pidgin/gtkxfer.c	Wed Jun 13 03:18:40 2018 +0000
@@ -256,18 +256,9 @@
 							 purple_xfer_get_size(xfer));
 
 	if (purple_xfer_is_completed(xfer)) {
-
-		GdkPixbuf *pixbuf = NULL;
-
-		pixbuf = gtk_widget_render_icon(xfer_dialog->window,
-										PIDGIN_STOCK_FILE_DONE,
-										GTK_ICON_SIZE_MENU, NULL);
-
 		gtk_list_store_set(GTK_LIST_STORE(xfer_dialog->model), &data->iter,
-						   COLUMN_STATUS, pixbuf,
+						   COLUMN_STATUS, NULL,
 						   -1);
-
-		g_object_unref(pixbuf);
 	}
 
 	if (purple_xfer_get_xfer_type(xfer) == PURPLE_XFER_TYPE_RECEIVE) {
@@ -594,7 +585,7 @@
 	/* Transfer Type column */
 	renderer = gtk_cell_renderer_pixbuf_new();
 	column = gtk_tree_view_column_new_with_attributes(NULL, renderer,
-				"pixbuf", COLUMN_STATUS, NULL);
+				"icon-name", COLUMN_STATUS, NULL);
 	gtk_tree_view_column_set_sizing(GTK_TREE_VIEW_COLUMN(column),
 									GTK_TREE_VIEW_COLUMN_FIXED);
 	gtk_tree_view_column_set_fixed_width(GTK_TREE_VIEW_COLUMN(column), 25);
@@ -868,7 +859,7 @@
 {
 	PidginXferUiData *data;
 	PurpleXferType type;
-	GdkPixbuf *pixbuf;
+	const gchar *icon_name;
 	char *size_str, *remaining_str;
 	char *lfilename, *utf8;
 
@@ -889,11 +880,7 @@
 	size_str      = purple_str_size_to_units(purple_xfer_get_size(xfer));
 	remaining_str = purple_str_size_to_units(purple_xfer_get_bytes_remaining(xfer));
 
-	pixbuf = gtk_widget_render_icon(dialog->window,
-									(type == PURPLE_XFER_TYPE_RECEIVE
-									 ? PIDGIN_STOCK_DOWNLOAD
-									 : PIDGIN_STOCK_UPLOAD),
-									GTK_ICON_SIZE_MENU, NULL);
+	icon_name = (type == PURPLE_XFER_TYPE_RECEIVE ?  "go-down" : "go-up");
 
 	gtk_list_store_append(dialog->model, &data->iter);
 	lfilename = g_path_get_basename(purple_xfer_get_local_filename(xfer));
@@ -901,7 +888,7 @@
 	g_free(lfilename);
 	lfilename = utf8;
 	gtk_list_store_set(dialog->model, &data->iter,
-					   COLUMN_STATUS, pixbuf,
+					   COLUMN_STATUS, icon_name,
 					   COLUMN_PROGRESS, 0,
 					   COLUMN_FILENAME, (type == PURPLE_XFER_TYPE_RECEIVE)
 					                     ? purple_xfer_get_filename(xfer)
@@ -914,8 +901,6 @@
 
 	gtk_tree_view_columns_autosize(GTK_TREE_VIEW(dialog->tree));
 
-	g_object_unref(pixbuf);
-
 	g_free(size_str);
 	g_free(remaining_str);
 
@@ -959,7 +944,6 @@
 								PurpleXfer *xfer)
 {
 	PidginXferUiData *data;
-	GdkPixbuf *pixbuf;
 	const gchar *status;
 
 	g_return_if_fail(dialog != NULL);
@@ -983,22 +967,16 @@
 	update_detailed_info(dialog, xfer);
 	update_title_progress(dialog);
 
-	pixbuf = gtk_widget_render_icon(dialog->window,
-									PIDGIN_STOCK_FILE_CANCELLED,
-									GTK_ICON_SIZE_MENU, NULL);
-
 	if (purple_xfer_is_cancelled(xfer))
 		status = _("Cancelled");
 	else
 		status = _("Failed");
 
 	gtk_list_store_set(dialog->model, &data->iter,
-	                   COLUMN_STATUS, pixbuf,
+	                   COLUMN_STATUS, "dialog-error",
 	                   COLUMN_REMAINING, status,
 	                   -1);
 
-	g_object_unref(pixbuf);
-
 	update_buttons(dialog, xfer);
 }
 
@@ -1044,18 +1022,10 @@
 
 	if (purple_xfer_is_completed(xfer))
 	{
-		GdkPixbuf *pixbuf;
-
-		pixbuf = gtk_widget_render_icon(dialog->window,
-										PIDGIN_STOCK_FILE_DONE,
-										GTK_ICON_SIZE_MENU, NULL);
-
 		gtk_list_store_set(GTK_LIST_STORE(xfer_dialog->model), &data->iter,
-						   COLUMN_STATUS, pixbuf,
+						   COLUMN_STATUS, NULL,
 						   COLUMN_REMAINING, _("Finished"),
 						   -1);
-
-		g_object_unref(pixbuf);
 	}
 
 	update_title_progress(dialog);

mercurial