@b4n requested changes on this pull request.

LGTM apart from the comment


In addons/src/addons.c:

> -#if ! GTK_CHECK_VERSION(2, 10, 0)
 	gtk_widget_hide(check_systray);
-#endif

Looks backwards, should you also remove the body? I guess the idea was not to add the option when the feature wasn't supported, wasn't it?


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/1358/review/2106383013@github.com>