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.

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

Read indicators can intersect with last message in message bubble layout #21469

Closed
alyaeanyx opened this issue Mar 18, 2022 · 4 comments
Closed
Labels
A-Message-Bubbles good first issue Good for newcomers Help Wanted Extra attention is needed O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect

Comments

@alyaeanyx
Copy link

alyaeanyx commented Mar 18, 2022

Steps to reproduce

  1. Activate message bubble layout
  2. Join a room with more than ~6 people in it
  3. Write a message
  4. Wait until all participants have read your message
  5. Click on the read indicators to expand them

Outcome

What did you expect?

The read indicators should not intersect with the bubble of the last message, instead appearing entirely below it

What happened instead?

The read indicators overlap with the last message.
Screenshot from 2022-03-18 11-54-25

Operating system

NixOS unstable

Application version

Element version: 1.10.4, Olm version: 3.2.8

How did you install the app?

From the nixpkgs-unstable repository

Homeserver

matrix.org

Will you send logs?

Yes

@germain-gg germain-gg added S-Tolerable Low/no impact on users A-Message-Bubbles O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Mar 18, 2022
@andybalaam andybalaam added the good first issue Good for newcomers label Mar 29, 2022
@agrani29
Copy link

Hey @andybalaam can you please assign this issue to me? I am an outreachy applicant.

@andybalaam
Copy link
Contributor

Hi @agrani29 just a comment saying you are working on it is fine - please go ahead.

@kittykat kittykat added the Help Wanted Extra attention is needed label May 4, 2022
@kittykat
Copy link
Contributor

kittykat commented May 4, 2022

@agrani29 I'm going to unassign this issue as per our guidance on assigning issues. You are still most welcome to continue working on it and I look forward to seeing your contribution :)

If you have any technical questions about this issue, you can ask for help in #element-dev:matrix.org

@robintown
Copy link
Member

This is fixed as of matrix-org/matrix-react-sdk#8389

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Message-Bubbles good first issue Good for newcomers Help Wanted Extra attention is needed O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Development

No branches or pull requests

6 participants