@techee commented on this pull request.


> +             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 
https://github.com/geany/geany/pull/3849/commits/1931fcc2f4d5de46871c85a782903d8c4d41e699
 from https://github.com/geany/geany/pull/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 or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1341#discussion_r1592766769
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany-plugins/pull/1341/review/2043668...@github.com>

Reply via email to