Page MenuHomePhabricator

[web] Fix fetching messages in messages list
ClosedPublic

Authored by tomek on Oct 8 2024, 8:35 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 1:05 AM
Unknown Object (File)
Sun, Nov 10, 5:25 AM
Unknown Object (File)
Sun, Nov 10, 5:02 AM
Unknown Object (File)
Sun, Nov 10, 3:13 AM
Unknown Object (File)
Sat, Nov 9, 9:48 PM
Unknown Object (File)
Thu, Nov 7, 5:19 PM
Unknown Object (File)
Mon, Nov 4, 5:05 AM
Unknown Object (File)
Mon, Nov 4, 4:41 AM
Subscribers
None

Details

Summary

When there were more than 40 messages in a thick thread, fetching more messages sometimes resulted in fetching more batches at once and jumping back a multiple of 20 messages. This issue was caused by an old bugfix that was fixing an issue where an empty space was displayed after opening a thread, when it had more than 20 messages https://github.com/CommE2E/comm/commit/0639b7628833dd318f58fb7ed36538dac75336a4. The solution to this is to apply this bugfix only when we initially open the page - this can be tested by checking if a scroll position is 0.

https://linear.app/comm/issue/ENG-9097/fetching-new-messages-in-message-list-is-glitchy-when-it-happens

Test Plan

Created a thread with 200 messages. Tested on Chrome and Safari.

  1. Opened the web page and scrolled slowly near every 20th messages. The original issue happened about 50% of the times, so I made sure to test it a couple of times. Didn't experience any glitches.
  2. Zoomed out so that more than 20 messages can be seen at once and opened the web app. Immediately after opening the app more messages were fetched.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage