canjs / can-list-sort

Live-bound list sorting
MIT License
1 stars 0 forks source link

dont forget to implement #2399 #3

Closed pYr0x closed 7 years ago

pYr0x commented 8 years ago

a PR was merged against canjs 2.3.24. do not forget to implement this in this repo also Pass list items to .sort(fn) and don't set the comparator

justinbmeyer commented 8 years ago

@ccummings any progress on this?

phillipskevin commented 7 years ago

This has already been implemented: https://github.com/canjs/can-list-sort/blob/master/can-list-sort-test.js#L707. can-list-sort was originally created in October and the original PR was made back in May.