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

Colors of avatars are broken (after upgrade 1.11.14) #772

Open
estellecomment opened this issue Oct 10, 2023 · 5 comments
Open

Colors of avatars are broken (after upgrade 1.11.14) #772

estellecomment opened this issue Oct 10, 2023 · 5 comments

Comments

@estellecomment
Copy link
Contributor

Du à l'upgrade 1.11.44
C'est moins lisible qu'avant.

Expected / obtained :

image image

Edit : this is filed already in element-web : element-hq/element-web#26285, on 13 sep.

@areox-net c'est grave docteur ?

@areox-net
Copy link
Contributor

@estellecomment Le contraste n'est vraiment pas top en l'état. S'il l'ont listé chez eux ça va peut-etre etre corrigée rapidement. Sinon on va etre obligé de faire un patch CSS temporaire.

@estellecomment estellecomment changed the title Colors of avatars are broken Colors of avatars are broken (after upgrade 1.11.14) Oct 31, 2023
@odelcroi
Copy link
Member

odelcroi commented Nov 7, 2023

The responsible for this mess is inside a dependency of matrix-react, the implementation of the design system of element : https://github.com/vector-im/compound-web/blob/1cdf6bef3f78f3afa15b1ae8d8f00c19a7248fb1/src/components/Avatar/Avatar.tsx#L107

@odelcroi
Copy link
Member

odelcroi commented Nov 7, 2023

@estellecomment Le contraste n'est vraiment pas top en l'état. S'il l'ont listé chez eux ça va peut-etre etre corrigée rapidement. Sinon on va etre obligé de faire un patch CSS temporaire.

@areox-net Si tu arrives à trouver un fix CSS on peut l'embarquer. De ce que j'ai vu le css est "généré" et possède un id généré. Cela rend compliqué la customization. Un deuxieme regard peut aider

@estellecomment
Copy link
Contributor Author

@areox-net on doit maintenant pouvoir y arriver avec --cpd-color-bg-decorative-1 (2, 3, ...) et --cpd-color-text-decorative-1 au lieu de --username-colors_1 et --avatar-background-colors_1.
Dans https://github.com/tchapgouv/tchap-web-v4/blob/3e7f7cd43e1ba22d8ff9b387e330e1f218b60241/res/themes/tchap-light/css/_tchap_custom_vars.pcss
A tester, parce que je sais plus ce que font ces variables. Si on peut nettoyer celles qui servent à rien ca serait cool !

@areox-net
Copy link
Contributor

@estellecomment Merci, il faudrait que je fasse une PR de test pour voir.

@areox-net areox-net self-assigned this Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants