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.

Sign up for GitHub

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

Jump to bottom

Default avatar colors are not matching EleWeb nor EleiOS #4676

Open
bmarty opened this issue Dec 10, 2021 · 1 comment
Open

Default avatar colors are not matching EleWeb nor EleiOS #4676

bmarty opened this issue Dec 10, 2021 · 1 comment
Labels
A-User Avatar 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 Something isn't working: bugs, crashes, hangs and other reported problems X-Needs-Product Issue needs input from Product team Z-parity-with-web

Comments

@bmarty
Copy link
Member

bmarty commented Dec 10, 2021 •
edited
Loading

Element Web Element Android Element iOS
image image image

Maybe the algorithm has changed on web...

On Android this is computed here:
https://github.com/vector-im/element-android/blob/main/vector/src/main/java/im/vector/app/features/home/room/detail/timeline/helper/MatrixItemColorProvider.kt#L50
using these colors:
image
image
image

Source of truth on Element Android: https://www.figma.com/file/X4XTH9iS2KGJ2wFKDqkyed/Compound?node-id=1521%3A23026
@bmarty bmarty added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-parity-with-web S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-User Avatar labels Dec 10, 2021
@bmarty bmarty added the X-Needs-Product Issue needs input from Product team label Dec 10, 2021
@bmarty bmarty changed the title Default avatar color are not matching EleWeb Default avatar colors are not matching EleWeb nor EleiOS Dec 15, 2021
@gaelledel
Copy link

Yes we should aim to find the same colours for all platforms. It seems to be a sync issue between platforms. @gsouquet and @SBiOSoftWhare might be able to help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-User Avatar 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 Something isn't working: bugs, crashes, hangs and other reported problems X-Needs-Product Issue needs input from Product team Z-parity-with-web
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bmarty @gaelledel