-
Notifications
You must be signed in to change notification settings - Fork 979
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
Sort pinned messages by time of pinning #12402
Conversation
58a9343
to
66cb2bf
Compare
Jenkins BuildsClick to see older builds (9)
|
@briansztamfater can we rebase PR please after merging #12411? |
66cb2bf
to
4cdd5c2
Compare
Rebase done @churik |
@briansztamfater thank you, works fine! The only thing is that currently, the newest pinned messages are placed at the bottom of the list. On Desktop they are at the top. Could you change the order accordingly, please? Steps:
|
@qoqobolo I can easily change that! Will do and let you know |
@briansztamfater just a reminder of this one :) |
@qoqobolo sorry for the delay, changes are now done! |
39ea692
to
b1d2ba0
Compare
Thank you @briansztamfater! |
Signed-off-by: Brian Sztamfater <brian@status.im>
b1d2ba0
to
0acca32
Compare
fixes #12290
Summary
This PR adds sorting pinned messages by time of pinning and also removes separation of pinned messages in timeframes
Screenshots
Platforms
Functional
Steps to test
status: ready