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

Placeholder avatars are 'wrong' colour. #1345

Closed
ara4n opened this issue Jul 17, 2023 · 3 comments · Fixed by #1610
Closed

Placeholder avatars are 'wrong' colour. #1345

ara4n opened this issue Jul 17, 2023 · 3 comments · Fixed by #1610
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product

Comments

@ara4n
Copy link
Member

ara4n commented Jul 17, 2023

Steps to reproduce

Placeholder avatars (i.e. those for rooms without explicit avatars) don't match the colours used by Element Web (presumably we're hashing the wrong thing, or using the wrong hashing function). Plus the actual shades of colours don't match Element Web.

Outcome

What did you expect?

No cognitive dissonance due to inconsistent colours between platforms.

What happened instead?

Extreme cognitive dissonance, similar to seeing the word RED but painted green.

Your phone model

No response

Operating system version

No response

Application version

292

Homeserver

No response

Will you send logs?

No

@ara4n ara4n added the T-Defect label Jul 17, 2023
@stefanceriu stefanceriu added the S-Minor Impairs non-critical functionality or suitable workarounds exist label Jul 19, 2023
@stefanceriu
Copy link
Member

Implementation here

@pixlwave
Copy link
Member

Noting that the correct solution is being worked on as part of Compound and hopefully will get implemented next week.

@ara4n
Copy link
Member Author

ara4n commented Jul 24, 2023

also the colour of placeholder avatars in RRs doesn't always match that of the user in the room icon in a 1:1...

@Velin92 Velin92 linked a pull request Sep 1, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants