ds.sort syntax is ugly
Closed this issue · 1 comments
alexgraul commented
currently to set and run a sort comparator you need to do
ds.sort( { comparator: function(a,b) {...} });
The signature could be changed to ds.sort( comparator, options );
to be a big nicer and remove the need for the extra object most of the time.
iros commented
Since you can set comparator on instantiation AND set ds.comparator = function(a,b), in this signature comparator is optional. As such, it goes into the optional arguments which can have silent:true. If we make it a param then it might require us passing null as first arg to set a silent:true.