search

miguelcobain / ember-yeti-table

Yeti Table
https://miguelcobain.github.io/ember-yeti-table
MIT License
61 stars 15 forks source link

Merge sort implementation should be removed in favor of the built-in sort function #398

Open LuisAverhoff opened 3 years ago

LuisAverhoff commented 3 years ago

I know this repo is using merge sort because the built-in sort function provided by browsers in the past wasn't stable but that is no longer true. If you take a look in mdn, you will see that all major browsers, except ie11 of course, offers a stable sorting algorithm. I propose that we ditch the merge sort implementation in favor of the built-in sort function and allow users to polyfill a stable sorting algorithm in older browsers that don't support it.