Save an undo checkpoint before paste in insert mode

[the-mikedavis]

I often paste and find out that my clipboard didn't have what I expected on it. Previously, undoing the change (u in normal mode) would also undo everything else I added in insert mode which is usually too much.

[pascalkuthe]

I agree with this 👍 Should we also do the same thing for the various paste commands in insert mode? It would seem odd if bracketed paste does this but a cumst binding like a-v wouldn't

[the-mikedavis]

Yeah we could do this in helix_term::commands::paste_impl so it works for all paste commands in insert mode 👍. Pastes from other registers should be fine to have undo checkpoints for too (maybe you hit the wrong register before pasting with C-r for example).

[pascalkuthe]

A I keep forgetting about c-r, even tough it's such a nice binding 😅 Considering that c-r is a default binding it makes even more sense to create a checkpoint here 👍

[archseer]

Somewhat related to #7226 ? Should merge both in tandem

[the-mikedavis]

Not exactly, this one only has to do with insert mode behavior rather than mode switching, similar to #7747.

I've been meaning to take a look at #7226 and figure out what's going on in the mode-switching though, I'll give that a look soonish

[pascalkuthe]

e2a8774 from that PR is actually almost the same as the original version of this PR I think?

The changes to :write are probably orthogonal

[the-mikedavis]

Ah yeah actually I didn't notice that part. That part could be dropped after this change. It's saving after the paste which is a little different behavior-wise. I believe we want to save before the paste though not after.