search

jeancroy / fuzz-aldrin-plus

Sublime text like fuzzy filtering - compatible with atom/fuzzaldrin
MIT License
269 stars 21 forks source link

Match for full word provides different result #9

Closed TimvdLippe closed 8 years ago

TimvdLippe commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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