[mini.surround] Add cursor position to jumplist before mini.surround motions

[mantzu132]

mini.surround motions like sF(, sf{, etc. move the cursor to the target delimiter, but do not record the current position in the jumplist.

Default Vim motions like [{, [(, ]}, ]) do add to the jumplist when the target bracket is more than one line away. This means users can easily return to their previous location with <C-o> or <C-i>.

mini.surround's behavior is inconsistent with this.

I wonder if it would be a good idea to mirror what Vim motions do?

[echasnovski]

It might make sense indeed. There was a similar suggestion for 'mini.ai' (#1802) and the initial reasoning applies to 'mini.surround'. But as MiniAi.move_cursor() adds to jumplist, it might be a good idea to also make MiniSurround.find() do the same. I'll take a look.

[mantzu132]

Thanks for the fix :) You rock. 👍