@b4n requested changes on this pull request.
LGTM apart from the comment
> -#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 or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1358#pullrequestreview-2106383013
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany-plugins/pull/1358/review/[email protected]>
