Search without sorting

Hi there,

I have a large select with years. Let’s say 1900 until 2018. And add them with setChoices(). The items are already sorted so they do not need to be sorted again.

Two things:

1. If I add the years as integers the search is not giving any results. If I add them as strings it is working. Not a big deal and that is not what this issue is about.
2. If I search with string 20 the first result is 2009 and after that is is nicely sorted as expected: 2018, 2017, 2016, 2015, etc. The same thing happens when I search on 19: the first record is 1949 and after that the results are 1999, 1998 is missing (is somewhere below) 1997, 1996, 1995, 1994, etc.

I can’t see why this is happening. As far as I can see the results from fuse are correctly sorted (read: unchanged).

new Choices(document.getElementById('select'), {
          shouldSort: false,
          silent: true,
          searchResultLimit: 100,
          itemSelectText: '',
          fuseOptions: {
              shouldSort: false,
              includeScore: false,
              includeMatches: false,
              threshold: 0,
              location: 0,
              distance: 100,
              maxPatternLength: 32,
              minMatchCharLength: 1,
              keys: ["value"]
          }
});

See also this example https://plnkr.co/edit/d1hsGPour408YC5OqVSf?p=preview v3.0.3

Is there any way to just keep the sorting as it is?

[khawkins98]

We’re still on v2.7.8, but here’s an edit we’re thinking about making to our environment (eventually doing a PR):

1. Add a new option secondarySortByAlpha
2. Modifying sortByScore to be:

  var sortByScore = exports.sortByScore = function sortByScore(a, b) {
    if (this.config.secondarySortByAlpha) {
      if (Math.round(a.score * 100) / 100 == Math.round(b.score * 100) / 100) {
        return sortByAlpha(a,b);
      }
    }
    return a.score - b.score;
  };

So, here, in cases of a close match (two decimal points) they would be sorted alphabetically – which I imagine would do similarly for number labels.

Before:

After:

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.