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

Match for full word provides different result #9

Closed
TimvdLippe opened this issue Nov 11, 2015 · 4 comments
Closed

Match for full word provides different result #9

TimvdLippe opened this issue Nov 11, 2015 · 4 comments

Comments

@TimvdLippe
Copy link

When I type pe, I expect the snippet with acronym pe to show up first. However, it shows pes as first one.

These snippets are from https://github.com/robdodson/Atom-PolymerSnippets

image

@jeancroy
Copy link
Owner

Thank you for the reports.

The issue is that fuzzaldrin-plus has been integrated into atom first and foremost to test how it behave.
So it's not everywhere. Separately autocomplete-plus is modular and separate provider can do the job of ordering things.

As far as this library is concerned, it return the correct order.
img

One possible cause is that.

Ie the default search for external provider is a mix between keeping original ordering and using quality information from fuzzaldrin. This is done in case provider have external information on how thing should be ordered.

@TimvdLippe
Copy link
Author

Okay thanks for the explanation @jeancroy . So this is probably an issue with multiple providers you say? Is there any way I can fix it or do I just have to deal with it?

@jeancroy
Copy link
Owner

I'm thinking that if you edit the source snippets so PE appears before PES it should work.
(it looks like a bunch of separate file, maybe renaming the pes file so it appear after in alphabetical order should work)

This could probably be fixed, but it would be in autocomplete-plus.
We would then have to review the expectation of remain original order.
I'm going to close this issue, feel free to post more question or update.

@jeancroy
Copy link
Owner

On another hand, fixing this should place PE above PES before we receive it.

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

No branches or pull requests

2 participants