@techee commented on this pull request.


In projectorganizer/src/prjorg-goto-panel.c:

> +		gtk_icon_size_lookup(GTK_ICON_SIZE_MENU, &x, &dummy);
+	}
+	return gtk_icon_theme_load_icon(icon_theme, icon_name, x, 0, NULL);
+}
+
+
+static GdkPixbuf *get_icon_pixbuf(gint icon)
+{
+	if (!geany_icons[_ICON_CLASS].pixbuf)
+	{
+		guint i;
+		for (i = 0; i < G_N_ELEMENTS(geany_icons); i++)
+			geany_icons[i].pixbuf = get_tag_icon(geany_icons[i].icon_name);
+	}
+
+	if (icon < _N_ICONS)

I'd prefer if geany/geany@1931fcc from geany/geany#3849 could be merged and used here. While the icons can be accessed this way, if someone reorganizes something in Geany, it will break. So better have some proper API for accessing these icons.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany-plugins/pull/1341/review/2043668149@github.com>