Always reset backgrounds first on any dialog response, then set again if necessary. When showing a dialog this was only done when using the document selected text, but the document could have changed e.g. to contain the previously unmatched search text, yet the background remained. Also the Find All responses didn't clear the background even when successful. This fixes those situations.
Set search entry background when there's a regex or escape error. Set replace text entry background when there's an escape error. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3828
-- Commit Summary --
* Make Find set entry background on error * Make Replace set entry backgrounds on error * Always reset entry backgrounds on dialog response
-- File Changes --
M src/search.c (53) M src/ui_utils.c (5)
-- Patch Links --
https://github.com/geany/geany/pull/3828.patch https://github.com/geany/geany/pull/3828.diff
github-comments@lists.geany.org