Web: Scroll position is not at the new message indicator on switching channels with unreads between 30-60

Description

Happens on 5.12v and 5.13v

Repo:
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

QA Test Steps

see description

Activity

Show:
Lindy Isherwood
July 24, 2019, 8:54 AM

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)

Lindy Isherwood
July 24, 2019, 9:43 AM

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.

Sudheer Timmaraju
July 24, 2019, 10:11 AM

Ticket for new messages indicator issue on first visit to a channel

Lindy Isherwood
July 24, 2019, 5:13 PM

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.

Lindy Isherwood
July 24, 2019, 5:15 PM

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.

Done

Mana

None

Assignee

Sudheer Timmaraju

QA Assignee

Lindy Isherwood

Reporter

Sudheer Timmaraju

Epic Link

None

Fix versions

Mattermost Team

Sustained Engineering

Sprint

None

Labels

QA Testing Areas

Messages

GitHub Issue

None

Components

None

Severity

None
Configure