Timeline events should use dates to order rather than rely on their position in an array.
CodeSpent opened this issue · comments
Patrick Hanford commented
Considering this component acts as a timeline and date ranges are provided, this could be used for sorting the timeline elements in chronological order.
I was able to get this working in my own project, and will look at opening a PR for this.
Joe Cox commented
I think this would be good as an optional configuration, but it should also be able to sort by the original position in array.
Stéphane Monnot commented
I think this would be better to sort outside the component, in javascript on client side before passing props to Timeline component or on server side.
Clément BENOIT commented
What would happen if the dates overlapped?