-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Date separator is wrong on inverted list #124
Comments
@SebastienBtr any comments on this bug? |
it seems to appear only for the first message, I didn't have time to check the code so I don't know exactly why it happens. I'll check that when I have more time. There is multiple issues with the date separator apparently, I'd honestly suggest to maybe not use it and wait for the v2. Anyway your issue is maybe quick to fix so I'll keep you updated. |
Ok, thanks!
… El 18 ago 2020, a las 12:17, SebastienBtr ***@***.***> escribió:
it seems to appear only for the first message, I didn't have time to check the code so I don't know exactly why it happens. I'll check that when I have more time. There is multiple issues with the date separator apparently, I'd honestly suggest to maybe not use it and wait for the v2. Anyway your issue is maybe quick to fix so I'll keep you updated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#124 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMALDHRASM5WF4OK3EU7ATSBJIKNANCNFSM4PXNKCFQ>.
|
You can also use a sorting function based on date to fix it, with something like this: messages.sort((messageA, messageB) => |
Describe the bug
When inverted: true, date separator doesn't work as expected.
To Reproduce
Expected behavior
Three different date separators, only found 2 and with wrong date
Screenshots
First screenshot, inverted: true, where the problem occurs.
Second screenshot, inverted: false, all ok.
The text was updated successfully, but these errors were encountered: