remplacement de twitter par X

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir

https://zestedesavoir.com

Other

268 stars 161 forks source link

remplacement de twitter par X #6627

Closed leroivi closed 3 months ago

leroivi commented 4 months ago

Ajout des icones X, suppression des icones twitter, remplacement des mentions twitter par X

Fix #6600

Contrôle qualité

Lancer le serveur make zmd-start && make run
Ouvrir la page d'acceuil
Vérifier que le bouton X est présent dans le pied de page et réagisse au survol de la souris
Aller sur une page de contenu
Vérifier que le bouton de partage X est présent

coveralls commented 4 months ago

coverage: 88.878%. remained the same when pulling 3aa53f0e62c8768afb35c5e7a94e103fccf24610 on leroivi:replace_twitter_X into d700a627f081748b4a480160335b9057e61027b1 on zestedesavoir:dev.

leroivi commented 3 months ago

Ok pour le changement d'icône (d'où est-ce qu'elle vient ?).

Le svg vient de leur brand-toolkit et les icones sont des dérivés de ce svg que j'ai fait pour mettre nos couleurs et notre résolution.

Ok pour le reste.
Pour le meta je savais pas que c'était la doc qui donnais les noms, désolé.
Et pour les mentions dans le code j'ai hésité parce que c'est chiant d'avoir qu'un caractère, mais je me suis dit que dorénavant twitter n'existait plus et il faut s'y faire.

Je vous laisse prendre une décision sur les mentions que mentionne Amaury, ça ne me gênera pas de revert si c'est ce qu'il faut.

philippemilink commented 3 months ago

Je vous laisse prendre une décision sur les mentions que mentionne Amaury, ça ne me gênera pas de revert si c'est ce qu'il faut.

Fais comme Amaury propose, j'ai l'impression qu'une fois les corrections d'Amaury appliquées, les x restants me gêneront moins 🙂