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 receipt sometimes fails to fall down #3976

Closed
uhoreg opened this issue May 21, 2017 · 7 comments
Closed

Read receipt sometimes fails to fall down #3976

uhoreg opened this issue May 21, 2017 · 7 comments
Labels
A-Read-Receipts O-Uncommon Most users are unlikely to come across this or unexpected workflow P3 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect X-Cannot-Reproduce

Comments

@uhoreg
Copy link
Member

uhoreg commented May 21, 2017

Description

Sometimes a read receipt fails to drop down from its old position to its new position.

Steps to reproduce

  • Sit around patiently watching read receipts drop

Seriously, though, I have no idea how to reproduce it. It seems to happen intermittently, and I haven't notice any pattern, other than the instances that I've checked seem to all be for users with no avatar set, but that could just be coincidental.

This screen shot shows the bug:
image

Note the blank space in the middle of the bottom row of RRs. That space opened up to allow the "S" beside Matthew's RR in the top row to drop down into it. However, it's just sitting there and not dropping. At the bottom, you can see the inspector. The highlighted element is the containing span for that RR, and you can see that its style attribute has top: -170px. It seems like the animation decided to just quit part way through.

The only possibly relevant thing that I can see in the browser's console log is some 404 errors, but I don't know if it's connected at all.

WQyplciIqkMkuusGFQvcqgxb Failed to load resource: the server responded with a status of 404 (Not Found)
https://matrix.org/_matrix/media/v1/thumbnail/matrix.org/WQyplciIqkMkuusGFQvcqgxb?width=30&height=30&method=crop Failed to load resource: the server responded with a status of 404 (Not Found)
WQyplciIqkMkuusGFQvcqgxb Failed to load resource: the server responded with a status of 404 (Not Found)

Log: not sent

Version information

  • Platform: screenshot is from web, but I've also seen it in desktop

For the web app:

  • Browser: Chrome 56.0.2924.87
  • OS: Debian
  • URL: riot.im/staging (0.9.10-rc.1)

For the desktop app:

  • OS: Debian
  • Version: 0.9.9
@lampholder lampholder added T-Defect S-Tolerable Low/no impact on users S-Major Severely degrades major functionality or product features, with no satisfactory workaround ui/ux X-Cannot-Reproduce S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels May 23, 2017
@lampholder
Copy link
Member

Ummed and erred about major/minor on this - I think it looks super glitchy so it's important to tackle, but I've never noticed it personally (making me quite possibly erroneously conclude that it is not common to see), hence minor.

If lots of people are seeing and noticing this then we could bump it to higher severity.

@lampholder lampholder added the P1 label May 23, 2017
@uhoreg
Copy link
Member Author

uhoreg commented May 23, 2017

Yeah, it happens quite rarely, so minor is probably about right.

@uhoreg uhoreg added P3 and removed P1 labels Aug 22, 2017
@uhoreg
Copy link
Member Author

uhoreg commented Aug 22, 2017

Dropping to p3 because I haven't seen this happen for a long time. I'll keep an eye on this for a while longer, and if it still doesn't happen, I'll just close the issue.

@turt2live
Copy link
Member

turt2live commented Aug 22, 2017

I honestly see this a few times a day, but it doesn't bother me too much.

@uhoreg
Copy link
Member Author

uhoreg commented Aug 26, 2017

I also just saw it happen again (in 0.12.2).

@t3chguy
Copy link
Member

t3chguy commented Aug 26, 2017

as did I @uhoreg
#riot:matrix.org ?

@jryans jryans removed the Z-UI/UX label Mar 9, 2021
@kittykat kittykat added the O-Uncommon Most users are unlikely to come across this or unexpected workflow label Aug 17, 2021
@kittykat
Copy link
Contributor

#9769 should have covered all the fixes needed for this issue, please comment if it's still present for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Read-Receipts O-Uncommon Most users are unlikely to come across this or unexpected workflow P3 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect X-Cannot-Reproduce
Projects
None yet
Development

No branches or pull requests

6 participants