On Tue, 21 Sep 2010 15:56:31 +1000 Lex Trotman elextr@gmail.com wrote:
IMHO, find and replace is such a core component of a text editor, it has to be done right. Both find selected and find next should be the same key. If something is selected then pressing the find key should search for it. If nothing is selected, it should search for the word that was last time time highlighted and searched for.
I don't think they should be the same keybinding because you may want to repeat the last search even though you have text selected. Lex's solution is good though.
Hear is a quick and dirty patch.
Committed a similar change.
Someone might consider if making find_again in callbacks.c global is the right way to go or if it should be in a utils_... file instead.
I put it in search.c. utils.c should be for general C stuff, non-GUI.
Regards, Nick