When the "Wrap search and find again?" dialog appears, it makes more sense to wrap instead of cancel the dialog by default (the dialog can always be canceled by Escape).
The implementation of this patch more or less duplicates the code of dialogs_show_question_full() and makes the Find button default. While it would be possible to add one more parameter to dialogs_show_question_full() controlling this behavior, there's currently no other use of this in Geany (in all other cases, the action button performs a destructive action so Cancel is default) so I believe having this local implementation of the dialog is a better option not complicating other uses. Fixes #1192. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/4042 -- Commit Summary -- * Make "Find" the default button in the wrap search dialog -- File Changes -- M src/document.c (39) -- Patch Links -- https://github.com/geany/geany/pull/4042.patch https://github.com/geany/geany/pull/4042.diff -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/4042 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/[email protected]>
