Use different widgets for 'i' and 'a'

[eleanor-clifford]

The current implementation of zvm_enter_insert_mode breaks if you don't use 'i' and 'a' to trigger it (which is a problem for me, as I use a non-standard keyboard). This PR replaces zvm_enter_insert_mode with zvm_enter_insert_mode_before and zvm_enter_insert_mode_after, for 'i' and 'a' respectively.

I'm aware that this will break functionality for anyone who has remapped or explicitly mapped zvm_enter_insert_mode, if you'd like we can leave the original implementation in, with a warning to change the remaps to the new functions.

[jeffreytse]

Thank you, I understand your concern, I will do some revision for better compatibility on this issue.

[jeffreytse]

For now, we can settle this issue in below snippet:

# The plugin will auto execute this zvm_after_lazy_keybindings function
function zvm_after_init() {
  zvm_bindkey vicmd ';' zvm_enter_insert_mode i
  zvm_bindkey vicmd ':' zvm_enter_insert_mode a
}

And welcome to star this project for further updates in the future. : )

[jeffreytse]

This revision allows users to do some complexed bindings.

[eleanor-clifford]

This way of doing it looks good for compatibility! Looks like you missed a few things though, i don't think this will work in the current state (although haven't tested)

[eleanor-clifford]

Oh, i just saw that you fixed it on the latest commit in master, nevermind!