I'm trying to sort a list alphabetically only using javascript.

I've already none a similar function using jquery like this:

$('.articles').append($('#articles-non-attached li'));
        var alphabeticallyOrderedDivs = $('.articles li').sort(function(a, b) {
            return String.prototype.localeCompare.call($(a).data('title').toLowerCase(), $(b).data('title').toLowerCase());
        });

But now I need to use just Javascript.

So far I've got:

var stores_li = document.querySelectorAll('.store-list-item');

//stores_li.sort();

[].slice.call(stores_li).sort(function(a, b) {
    var textA = a.getAttribute('data-title').toLowerCase()
    var textB = b.getAttribute('data-title').toLowerCase()
    return (textA < textB) ? -1 : (textA > textB) ? 1 : 0;
    //return String.prototype.localeCompare.call(textA,textB);

});

<ul>
  <li class="store-list-item" data-title="a">Test 1</li>
  <li class="store-list-item" data-title="c">Test 3</li>
  <li class="store-list-item" data-title="b">Test 2</li>
  <li class="store-list-item" data-title="d">Test 4</li>
</ul>

I'm trying to sort a list alphabetically only using javascript.

I've already none a similar function using jquery like this:

$('.articles').append($('#articles-non-attached li'));
        var alphabeticallyOrderedDivs = $('.articles li').sort(function(a, b) {
            return String.prototype.localeCompare.call($(a).data('title').toLowerCase(), $(b).data('title').toLowerCase());
        });

But now I need to use just Javascript.

So far I've got:

var stores_li = document.querySelectorAll('.store-list-item');

//stores_li.sort();

[].slice.call(stores_li).sort(function(a, b) {
    var textA = a.getAttribute('data-title').toLowerCase()
    var textB = b.getAttribute('data-title').toLowerCase()
    return (textA < textB) ? -1 : (textA > textB) ? 1 : 0;
    //return String.prototype.localeCompare.call(textA,textB);

});

<ul>
  <li class="store-list-item" data-title="a">Test 1</li>
  <li class="store-list-item" data-title="c">Test 3</li>
  <li class="store-list-item" data-title="b">Test 2</li>
  <li class="store-list-item" data-title="d">Test 4</li>
</ul>

• javascript

• what is the problem? – brk Commented Oct 3, 2017 at 14:23
• It's not sorting alphabetically. It should return Test 1, Test 2, Test 3, Test 4. But it's returning test 1, test 3, test 2, test 4. – Cátia Rodrigues Commented Oct 3, 2017 at 14:26

2 Answers 2

You need to append the newly sorted items.

var stores_li = document.querySelectorAll('.store-list-item');

[].slice.call(stores_li).sort(function(a, b) {
    var textA = a.getAttribute('data-title').toLowerCase()
    var textB = b.getAttribute('data-title').toLowerCase()
    return (textA < textB) ? -1 : (textA > textB) ? 1 : 0;
})
  .forEach(function(el) {el.parentNode.appendChild(el)});

<ul>
  <li class="store-list-item" data-title="a">Test 1</li>
  <li class="store-list-item" data-title="c">Test 3</li>
  <li class="store-list-item" data-title="b">Test 2</li>
  <li class="store-list-item" data-title="d">Test 4</li>
</ul>

This is because the native .sort() method isn't a DOM method; it just sorts any arbitrary list of Array items. So by appending the newly sorted item, it mits the new element order to the DOM.

Note that the simple solution above assumes they all share the same parent.

Here's a version with modern syntax and methods:

var stores_li = document.querySelectorAll('.store-list-item');

Array.from(stores_li).sort((a, b) =>   
  a.dataset.title.toLowerCase().localeCompare(b.dataset.title.toLowerCase())
)
  .forEach(el => el.parentNode.appendChild(el));

<ul>
  <li class="store-list-item" data-title="a">Test 1</li>
  <li class="store-list-item" data-title="c">Test 3</li>
  <li class="store-list-item" data-title="b">Test 2</li>
  <li class="store-list-item" data-title="d">Test 4</li>
</ul>

var stores_li = document.querySelectorAll('.store-list-item');
var sorted = [].slice.call(stores_li).sort(function(a, b) {
    var textA = a.dataset.title.toLowerCase();
    var textB = b.dataset.title.toLowerCase();

    return textA.localeCompare(textB);
});