Skip to content
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

Fix completion minimum pattern length #468

Merged
merged 1 commit into from
Jan 18, 2016
Merged

Conversation

tony
Copy link
Contributor

@tony tony commented Jan 18, 2016

See also: https://git.io/vzWX4
Related: #463

@xaizek
Copy link
Collaborator

xaizek commented Jan 18, 2016

Thanks, although I doubt it's related to #463, no mentions of neovim there and this code is run only there.

xaizek added a commit that referenced this pull request Jan 18, 2016
Fix completion minimum pattern length
@xaizek xaizek merged commit 2289f34 into xavierd:master Jan 18, 2016
@expipiplus1
Copy link

With this change I'm seeing the popup menu constantly which isn't particularly pleasant. Is this something other's are experiencing or should I start chopping up my config to discover what's up?

@tony
Copy link
Contributor Author

tony commented Jan 19, 2016

@expipiplus1 We can revert it if it's causing an issue.

No offense taken here, autocompletion in vim is lost on me. Its so convoluted there is little way to determine what is coming from what.

@expipiplus1
Copy link

@tony The latest bit of chatter in #464 is about making this behave a little better.

I couldn't agree more about completion in vim being a mess, it's such a shame.

Perhaps @Shougo could shed some light here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants