mirrors/element-web
1
0
Fork 0
mirror of https://github.com/element-hq/element-web synced 2024-11-26 11:15:53 +03:00
Code Issues Projects Releases Packages Wiki Activity

Don't form continuations on either side of a thread root (#8408)

Browse source
This commit is contained in:
Michael Telatynski 2022-04-26 09:30:36 +01:00 committed by GitHub
parent f85e178fc1
commit a70f11704f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 26 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/components/structures/MessagePanel.tsx
View file

@ -92,9 +92,13 @@ export function shouldFormContinuation(
mxEvent.sender.name !== prevEvent.sender.name || mxEvent.sender.name !== prevEvent.sender.name ||
mxEvent.sender.getMxcAvatarUrl() !== prevEvent.sender.getMxcAvatarUrl()) return false; mxEvent.sender.getMxcAvatarUrl() !== prevEvent.sender.getMxcAvatarUrl()) return false;
// Thread summaries in the main timeline should break up a continuation // Thread summaries in the main timeline should break up a continuation on both sides
if (threadsEnabled && prevEvent.isThreadRoot && if (threadsEnabled &&
timelineRenderingType !== TimelineRenderingType.Thread) return false; (mxEvent.isThreadRoot || prevEvent.isThreadRoot) &&
timelineRenderingType !== TimelineRenderingType.Thread
) {
return false;
}
// if we don't have tile for previous event then it was shown by showHiddenEvents and has no SenderProfile // if we don't have tile for previous event then it was shown by showHiddenEvents and has no SenderProfile
if (!haveRendererForEvent(prevEvent, showHiddenEvents)) return false; if (!haveRendererForEvent(prevEvent, showHiddenEvents)) return false;

23
test/components/structures/MessagePanel-test.js
View file

@ -674,6 +674,20 @@ describe('MessagePanel', function() {
describe("shouldFormContinuation", () => { describe("shouldFormContinuation", () => {
it("does not form continuations from thread roots", () => { it("does not form continuations from thread roots", () => {
const message1 = TestUtilsMatrix.mkMessage({
event: true,
room: "!room:id",
user: "@user:id",
msg: "Here is a message in the main timeline",
});
const message2 = TestUtilsMatrix.mkMessage({
event: true,
room: "!room:id",
user: "@user:id",
msg: "And here's another message in the main timeline",
});
const threadRoot = TestUtilsMatrix.mkMessage({ const threadRoot = TestUtilsMatrix.mkMessage({
event: true, event: true,
room: "!room:id", room: "!room:id",
@ -682,14 +696,15 @@ describe("shouldFormContinuation", () => {
}); });
jest.spyOn(threadRoot, "isThreadRoot", "get").mockReturnValue(true); jest.spyOn(threadRoot, "isThreadRoot", "get").mockReturnValue(true);
const message = TestUtilsMatrix.mkMessage({ const message3 = TestUtilsMatrix.mkMessage({
event: true, event: true,
room: "!room:id", room: "!room:id",
user: "@user:id", user: "@user:id",
msg: "And here's another message in the main timeline", msg: "And here's another message in the main timeline after the thread root",
}); });
expect(shouldFormContinuation(threadRoot, message, false, true)).toEqual(false); expect(shouldFormContinuation(message1, message2, false, true)).toEqual(true);
expect(shouldFormContinuation(message, threadRoot, false, true)).toEqual(true); expect(shouldFormContinuation(message2, threadRoot, false, true)).toEqual(false);
expect(shouldFormContinuation(threadRoot, message3, false, true)).toEqual(false);
}); });
}); });