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

1:1 room avatar not always updated with user's changes #4086

Closed
her001 opened this issue May 30, 2017 · 1 comment
Closed

1:1 room avatar not always updated with user's changes #4086

her001 opened this issue May 30, 2017 · 1 comment
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce

Comments

@her001
Copy link

her001 commented May 30, 2017

Description

In a 1:1 chat with no room avatar set, it appears that Riot will sometimes update the icon that is displayed with the room to match the avatar of the other person in the room, but not usually. This results in a lot of my rooms having an icon that is the generic letter on a flat colored background, even though the user has set an avatar for themselves since they have been invited.

Steps to reproduce

  • Invite someone to a new chat (without setting a room avatar)
  • Have that person change their avatar after joining
  • The room icon will usually stay the same as before it was changed

I expect for the room icon to update to the same as the person's avatar, for as long as the room does not have an avatar of its own. Unfortunately this might be harder to track down, as sometimes this does work as expected, but in my experience it normally does not.

Log: not sent (I don't think it would help)

screenshot_2017-05-30_11-16-48

Version information

  • Platform: web
  • Browser: Firefox 53.0.3
  • OS: Arch Linux
  • URL: riot.im/develop and riot.im/app

Possibly Related Issues

#3256 If you add a 3rd person to a room and then remove them, avatar doesn't return to 2nd person

#3768 allow the user to prefer no more than one avatar per person in a room

@lampholder lampholder added feature T-Defect X-Cannot-Reproduce S-Minor Impairs non-critical functionality or suitable workarounds exist P2 and removed feature labels May 31, 2017
@her001
Copy link
Author

her001 commented May 31, 2017

I just realized the pattern here. Those rooms that haven't changed both had tested integrations (adding bots then removing them later).

This issue is a duplicate of #3256 and can be closed. Sorry!

@t3chguy t3chguy closed this as completed May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce
Projects
None yet
Development

No branches or pull requests

3 participants