diff --git a/playwright/e2e/threads/threads.spec.ts b/playwright/e2e/threads/threads.spec.ts index 98039e1242..79f6000751 100644 --- a/playwright/e2e/threads/threads.spec.ts +++ b/playwright/e2e/threads/threads.spec.ts @@ -445,7 +445,7 @@ test.describe("Threads", () => { await expect(locator.locator(".mx_EventTile").last().getByText("Hello Mr. User")).toBeAttached(); }); - test("navigate through right panel", async ({ page, app, user }) => { + test("navigate through right panel", { tag: "@screenshot" }, async ({ page, app, user }) => { // Create room const roomId = await app.client.createRoom({}); await page.goto("/#/room/" + roomId); @@ -497,6 +497,9 @@ test.describe("Threads", () => { await expect( threadPanel.locator(".mx_EventTile_last").getByText("Hello again Mr. User in a thread"), ).toBeVisible(); + await expect(threadPanel).toMatchScreenshot("thread-panel.png", { + css: ".mx_MessageTimestamp { visibility: hidden !important; }", + }); const rightPanel = page.locator(".mx_RightPanel"); // Check that the threads are listed diff --git a/playwright/snapshots/threads/threads.spec.ts/thread-panel-linux.png b/playwright/snapshots/threads/threads.spec.ts/thread-panel-linux.png new file mode 100644 index 0000000000..3c78fa1679 Binary files /dev/null and b/playwright/snapshots/threads/threads.spec.ts/thread-panel-linux.png differ diff --git a/src/components/views/rooms/EventTile.tsx b/src/components/views/rooms/EventTile.tsx index a7fd7527f6..edab5bc947 100644 --- a/src/components/views/rooms/EventTile.tsx +++ b/src/components/views/rooms/EventTile.tsx @@ -360,6 +360,9 @@ export class UnwrappedEventTile extends React.Component // We only show the 'sent' receipt on the last successful event. if (!this.props.lastSuccessful) return false; + // Don't show this in the thread view as it conflicts with the thread counter. + if (this.context.timelineRenderingType === TimelineRenderingType.ThreadsList) return false; + // Check to make sure the sending state is appropriate. A null/undefined send status means // that the message is 'sent', so we're just double checking that it's explicitly not sent. if (this.props.eventSendStatus && this.props.eventSendStatus !== EventStatus.SENT) return false;