Happens on 5.12v and 5.13v
1. Have unreads between 30-60 in a channel
2. Switch to the channel with unreads
Expected: Load 30 posts and make another API call to load 30 more as all of the messages are unread. After loading the new 30 posts scroll position should be fixed wherever it is left at(If no scroll happened between the second API call then scroll position should be at the top of 30 posts).
Observed: UI loads 30 posts make another API call and keep the scroll position at the top of 30 posts
Reference for expected behaviour
Hi , tested on prev.test v5.13.2 RC1
Testing with 30, 40 and 50 posts, I do not see the 'new messages' indicator in the channel at all and had to scroll up to see it (see screenshots)
I think I might have found something - if I first add another test user to the channel I'm going to make the '/test posts' to, this seems to mess with the scroll position of the 'new messages' indicator.
I found this because I first tested by adding a test user to each of the channels I was going to post to. Then I went back to test without adding a test user to the channel first and then the scroll position is at the 'new messages' indicator.
I could be totally wrong - it might not be connected at all but that's what it looked like to me while testing.
(To be sure that this is somehow causing the behavior, I just went back again, created new channels, added the test user to the channels and then tested 30, 40 and 50 test posts and scroll position was not at the new messages indicator in any of those posts)
I'm sending this back to Triage to discuss in today's meeting. I did find an issue, as commented above, but Sudheer and I spent about an hour discussing/testing and it seems there is a long-standing issue of 'first visit' to a channel which messes with the 'new messages' indicator. Sudheer will create a follow-up ticket for this issue and also explain the issue in Triage.
Ticket for new messages indicator issue on first visit to a channel
As discussed in Triage, the issues in this ticket are fixed and the 'first visit to a channel' issue will be addressed in MM-17279.
Thanks ST, closing this ticket as working as expected. as per comments and discussion in Triage earlier.
QA test note: no new test required, covered in release testing. Labelled PR.