search

tchapgouv / tchap-web-v4

A Matrix web client for Tchap
https://www.tchap.gouv.fr/
GNU Affero General Public License v3.0
12 stars 5 forks source link

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

Open estellecomment opened 1 year ago

estellecomment commented 1 year ago

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

Expected / obtained :

image image

Edit : this is filed already in element-web : https://github.com/vector-im/element-web/issues/26285, on 13 sep.

@areox-net c'est grave docteur ?

areox-net commented 1 year ago

@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.

odelcroi commented 1 year ago

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 commented 1 year ago

@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 commented 7 months ago

@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 commented 7 months ago

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