-
Notifications
You must be signed in to change notification settings - Fork 204
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
Possible autocomplete+ provider priority issue #353
Comments
The snippets don't show reliably at the moment. That issue was reported : atom/autocomplete-snippets#46 They thought it was because of priority so they increased it to 2 atom/autocomplete-snippets#46 (comment) I'd rather not get into a priority war at the moment :) I'll see what other providers are doing 👍 |
They seem to prefer snippets on top i.e. none of these provide a priority : https://github.com/atom/autocomplete-atom-api/blob/master/lib/provider.coffee && https://github.com/atom/autocomplete-html/blob/master/lib/provider.coffee && https://github.com/atom/autocomplete-css/blob/master/lib/provider.coffee |
I'll make it 3. Since omnisharp did the same https://github.com/OmniSharp/omnisharp-atom/pull/305/files |
I can't imagine why anyone would prefer snippets above actual object completions, when in that context they're out of scope... Anyway, I'm glad this is solved, thanks! 😄 |
As far as this issue: atom/autocomplete-plus#465 is concerned, I talked to one of the devs who mentioned that setting the priority >2:
Should fix the issue.
I also want to say thanks for all your hard work @basarat it's much appreciated. You're probably getting tired of all the issues I keep creating 😜
The text was updated successfully, but these errors were encountered: