Development #50480
style pour les paragraphes de texte d'authentic
0%
Description
En testant je me rends compte que le message affiché en cas de robot est tout serré.
Pour les cellules de texte on a #36436 qui a augmenté le line-height, ça devrait s'appliquer aussi pour les bouts de texte d'authentic.
Fichiers
.png){kind=link}
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-scss-increase-default-line-height-in-authentic-texts.patch 0001-scss-increase-default-line-height-in-authentic-texts.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Ok très bien.
Mis à jour par Thomas Jund il y a plus de 3 ans
Toujours la même remarque pour ces patchs qui apportent de la spécificité, complique le code et au final fragilise le système, alors que ce devrait être une valeur héritée.
Typographiquement, line-height est lié aux autres attributs typo comme font-size et font-family parce que ça gère le rythme vertical et devrait être commun à tous corps de texte d'une plateforme.
Le corps de texte c'est p, li, dt,…
Et si on tape en dur un p dans une cellule json ou dans une cellule commentaire dans wcs, il doit être commun aux autres p constituant le corps de texte sans avoir besoin de se soucier à l'uniformiser.
Et maintenir une uniformité en dupliquant du code de tous les côtés, j'aime pas.
Bref, ma remarque est la même que là https://dev.entrouvert.org/issues/36436#note-4
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
Ok, donc même réponse, ça me rassure de ne pas taper partout mais je suis tout à fait d'accord avec le ticket qui voudra mettre en commun ça.
commit 625db6c9bea5d50ec4edca1d235cda09518449f5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Jan 24 12:17:23 2021 +0100
scss: increase default line height in authentic texts (#50480)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: increase default line height in authentic texts (#50480)