diff options
| author | HJ <spam@hjkos.com> | 2019-01-24 22:16:26 +0000 |
|---|---|---|
| committer | HJ <spam@hjkos.com> | 2019-01-24 22:16:26 +0000 |
| commit | 63df815dd946be74d23c0d5cc2832c7aa28fbda6 (patch) | |
| tree | 5ce2a6ab76a2096fec7d7608ea51a0238530b0c6 /src/modules/statuses.js | |
| parent | 03ffa7e84e648f1114605c338e6b40c214199e9d (diff) | |
| parent | 9d0d6b86c83a4fad4038077a65149f0f3677fb82 (diff) | |
Merge branch 'fixSortingSequentials' into 'develop'
Fix TL sorting
See merge request pleroma/pleroma-fe!474
Diffstat (limited to 'src/modules/statuses.js')
| -rw-r--r-- | src/modules/statuses.js | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/src/modules/statuses.js b/src/modules/statuses.js index 83cf7256..27d4e94c 100644 --- a/src/modules/statuses.js +++ b/src/modules/statuses.js @@ -81,7 +81,21 @@ const mergeOrAdd = (arr, obj, item) => { } } -const sortById = (a, b) => a.id > b.id ? -1 : 1 +const sortById = (a, b) => { + const seqA = Number(a.id) + const seqB = Number(b.id) + const isSeqA = Number.isNaN(seqA) + const isSeqB = Number.isNaN(seqB) + if (isSeqA && isSeqB) { + return seqA > seqB ? -1 : 1 + } else if (isSeqA && !isSeqB) { + return 1 + } else if (!isSeqA && isSeqB) { + return -1 + } else { + return a.id > b.id ? -1 : 1 + } +} const sortTimeline = (timeline) => { timeline.visibleStatuses = timeline.visibleStatuses.sort(sortById) @@ -239,7 +253,7 @@ const addNewStatuses = (state, { statuses, showImmediately = false, timeline, us processor(status) }) - // Keep the visible statuses sorted + // Keep the visible statuses sorted if (timeline) { sortTimeline(timelineObject) if ((older || timelineObject.minVisibleId <= 0) && statuses.length > 0) { |
