Improve backscroll loading
Categories
(Chat Core :: Matrix, task)
Tracking
(thunderbird102? fixed, thunderbird103 fixed)
People
(Reporter: freaktechnik, Assigned: freaktechnik)
References
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
patch
|
wsmwk
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
We currently show more events than we should be as backscroll. There are events leaking in from multiple sources, like decryption of older events.
We could also improve the backscroll loading by not displaying anything before a message that was sent by the current account.
Assignee | ||
Comment 1•3 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/42d76624220e
Improve Matrix catch up behavior. r=clokep
Assignee | ||
Comment 3•2 years ago
|
||
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Matrix will show messages the user has already seen again and again and some messages may appear out of order.
Testing completed (on c-c, etc.): Minimal testing on c-c, patch not built on beta (but linted fine)
Risk to taking this patch (and alternatives if risky): Honestly, this is rather risky. because it's a big patch with changes deep in the matrix sync logic. I'm not quite sure if this should even be uplifted, since matrix is "preliminarily supported", but in the same vein, as long as it doesn't make it much worse, matrix is only preliminary.
Comment 4•2 years ago
|
||
Comment on attachment 9280652 [details] [diff] [review]
bug1770459-beta.diff
[Triage Comment]
Approved for beta
Comment 5•2 years ago
|
||
bugherder uplift |
Thunderbird 102.0b5:
https://hg.mozilla.org/releases/comm-beta/rev/bb0f65091cfa
Description
•