Open audax opened 8 years ago
The color space should be defined in the core and used in both clients.
What about actually storing the color in the contact? We need it in increasingly more places (the avatar in the new chat ui is colored for example) and that would open the implementation for extensions like a color picker. ...an extension method to Contact could be an intermediate solution... Contact.color that resolves to an enum with the color codes defined above. We could then fallback to a random (or deterministic alias-based) pick from this enum and later switch that implementation to something that changes an actual property of that contact IF we need it ^^
edit: new title proposal define color space for contacts
@jmt85 commented on Mon Mar 07 2016
the avatar background should get a defined color space
The coler should calculate from e-mail and Name
the color space comes from Britta
@mr-gosh commented on Thu Mar 10 2016
it should be used the same color calculation in desktop and android - if possible...
@mr-gosh commented on Thu Aug 11 2016
we could check how other projects define there avatar background colors, or we just use https://github.com/wada811/Android-Material-Design-Colors/blob/master/library/src/main/res/values/colors.xml and in every "normal" color (not grey or blue-grey) we use the color-rooms 500 - 900 except yellow there it sould be 600 - 900.
@mr-gosh commented on Thu Aug 11 2016
@mr-gosh commented on Thu Aug 11 2016
please use this fixed list of colours for the avatar backgrounds - and the colour must be cross platform - so whatever you calculate should result in the same colour - no matter which platform i use...
@mr-gosh commented on Thu Aug 11 2016
or we go the signal way: https://github.com/WhisperSystems/Signal-Android/blob/ec63f7014e5e67993ac0793cc94de092d2b7eca6/src/org/thoughtcrime/securesms/contacts/avatars/ContactColors.java
@julianseeger commented on Thu Aug 11 2016
linked the android colors at https://github.com/Qabel/qabel-desktop/issues/408
@mr-gosh commented on Thu Sep 29 2016
@JoGir here they are