[Github-comments] [geany/geany] Define text color when setting dark background in find dialog (#2333)

[Colomban Wendling]

b4n approved this pull request.

LGTM.  Indeed, we didn't handle the case where the theme would set a specific foreground color for the selected text, separate from the one for unselected text.
I don't have such a theme, but the changes here look sensible to me, and should be harmless otherwise (but if someone changed the colors and now has to override this as well, but that's something I think we can live with).



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2333#pullrequestreview-295885623
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20191001/1a68ed69/attachment-0001.html>