Code review comment for lp:~jjardon/libappindicator/fix-820080

Revision history for this message
Ted Gould (ted) wrote :

While I agree that this is the "right" way to do it, GIcon support has been broken in GTK+ for a while and was causing upstream GNOME maintainers to claim our work was shoddy.

https://bugzilla.gnome.org/show_bug.cgi?id=614800

So that's why we stopped using it.

review: Disapprove

« Back to merge proposal