Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Redacted message in thread has read receipt *above* message instead of at the bottom #24808

Open
turt2live opened this issue Mar 12, 2023 · 1 comment
Open

Redacted message in thread has read receipt *above* message instead of at the bottom #24808

turt2live opened this issue Mar 12, 2023 · 1 comment
Labels
A-Read-Receipts A-Redaction A-Threads O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@turt2live
Copy link
Member

Steps to reproduce

  1. Have someone else send a message in the thread
  2. Have them redact it

Outcome

What did you expect?

The read receipt to be bottom-aligned with the event.

What happened instead?

image

Matthew's avatar is in line with his display name rather than the message

Operating system

Windows 10

Application version

Element Nightly version: 0.0.1-nightly.2023031001 Olm version: 3.2.12

How did you install the app?

The Internet

Homeserver

t2l.io

Will you send logs?

No
@t3chguy t3chguy added S-Minor Impairs non-critical functionality or suitable workarounds exist A-Read-Receipts A-Redaction O-Occasional Affects or can be seen by some users regularly or most users rarely A-Threads labels Mar 13, 2023
@luixxiul luixxiul mentioned this issue Mar 13, 2023
Closed
3 tasks
@luixxiul
Copy link

This should be fixed with matrix-org/matrix-react-sdk#10335 (this commit).

Before After
Read receipts (modern layout) Screenshot from 2023-03-13 18-41-09 Screenshot from 2023-03-13 18-42-02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Read-Receipts A-Redaction A-Threads O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix timestamp and read receipt group placement on ThreadView
3 participants
@turt2live @t3chguy @luixxiul