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

Plug mappings are ambiguous #9

Closed
guns opened this Issue Jun 30, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@guns
Contributor

guns commented Jun 30, 2012

Hello,

The plug mappings <Plug>NrrwrgnDo and <Plug>NrrwrgnDo1 are ambiguous
and cause a slight delay of timeoutlen ms, or a complete pause if set notimeout.

What is happening is that Vim sees that the first mapping may
potentially be a subset of the second mappings and waits for timeout
ms before executing the mapping. If notimeout is set, it waits for
user input to disambiguate the name clash. At this point the user can
actually trigger <Plug>NrrwrgnDo1 by pressing: <Leader>nr1!

I changed my local copy to simply rename the bang versions of the plugs
to <Plug>NrrwrgnBangDo and <Plug>VisualNrrwBangRgn to resolve.

BTW, this is a separate issue, but the new mapping <Leader>Nr does not
reuse the same window like :NR!, as it seems intended to do.

Thanks for the software!

@chrisbra

This comment has been minimized.

Show comment
Hide comment
@chrisbra

chrisbra Jun 30, 2012

Owner

Thanks for the detailed feedback. I fixed it and it should work now.

Owner

chrisbra commented Jun 30, 2012

Thanks for the detailed feedback. I fixed it and it should work now.

@guns

This comment has been minimized.

Show comment
Hide comment
@guns

guns Jun 30, 2012

Contributor

Who is Peng Su?

:)

  • Sung Pae
Contributor

guns commented Jun 30, 2012

Who is Peng Su?

:)

  • Sung Pae
@chrisbra

This comment has been minimized.

Show comment
Hide comment
@chrisbra

chrisbra Jul 1, 2012

Owner

Sorry, I'll update it in the documentation.

Owner

chrisbra commented Jul 1, 2012

Sorry, I'll update it in the documentation.

@guns

This comment has been minimized.

Show comment
Hide comment
@guns

guns Jul 1, 2012

Contributor

Oh please, don't bother. That's very polite of you, but very unnecessary!

Cheers

Contributor

guns commented Jul 1, 2012

Oh please, don't bother. That's very polite of you, but very unnecessary!

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment