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

RR animations no longer animate from the right point. #27504

Closed
ara4n opened this issue May 29, 2024 · 2 comments
Closed

RR animations no longer animate from the right point. #27504

ara4n opened this issue May 29, 2024 · 2 comments
Labels
A-Read-Receipts O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression

Comments

@ara4n
Copy link
Member

ara4n commented May 29, 2024

Steps to reproduce

  1. Send some messages to a DM
  2. Watch as the other user reads them.
  3. The starting point of their RR animations is from some point in the past (e.g. when they last sent a message, or the most recently read message when you opened the room, or something)
  4. As a result, the RR animations pop: rather than interpolating from their current position to the target position, they pop upwards by several messages and then interpolate to the correct target. This gives the impression of them "dropping out of the sky" rather than "reading the next msg".

This is a regression.

Outcome

What did you expect?

Smooth RR animations from current position to target position

What happened instead?

Popping RR animation from incorrect stale position to target position.

Operating system

No response

Application version

Element Nightly version: 2024052901 Crypto version: Rust SDK 0.7.0 (068a0af), Vodozemac 0.6.0

How did you install the app?

No response

Homeserver

No response

Will you send logs?

No

@ara4n ara4n added the T-Defect label May 29, 2024
@dosubot dosubot bot added A-Read-Receipts S-Minor Impairs non-critical functionality or suitable workarounds exist labels May 29, 2024
@dbkr dbkr added X-Regression O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Jun 10, 2024
@ara4n
Copy link
Member Author

ara4n commented Jun 21, 2024

rr.brokenness.mov

:(

@ara4n
Copy link
Member Author

ara4n commented Sep 4, 2024

dup #27916

@ara4n ara4n closed this as completed Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Read-Receipts O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression
Projects
None yet
Development

No branches or pull requests

2 participants