-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add backward jump mapping #13
Conversation
Thanks for implementing it, I had some trouble with muliple choice placeholders. Does it work for you with a snippet such as?
In my case (using the following mappings:
) jumping forward skips the BAZ placeholder so it jumps from FOO to BAR |
It is my mistake. I understood the reason. |
@pwntester I've fixed the issue. Please check it if you have time. It has breaking change, please check https://github.com/hrsh7th/vim-vsnip/pull/13/files#diff-4e12c6a37ff2cbb2c93d1b33324a6051R42 before check. |
Thanks, Tried with:
and got the same problems. Is that a correct config? |
@pwntester Ah, Sorry. The jump sequence is right. Your config is right. P.S. If you want |
oh my bad, I had incorrect order in placeholders. Now its working as expected, thanks! The only odd thing is that first time that I get to the multi-option placeholder (2 after fixing order), the menu is displayed and immediatly closed. If then I move to 3rd placeholder and then back to the 2nd one, the menu is displayed and it stays. Do you get the same behaviour? |
You use auto-completion engine, so pum is overwitten by it. |
the choice problemcis not related this PR. |
No description provided.