Development #36436
divers, augmenter le line-height par défaut des cellules texte
0%
Description
Aujourd'hui on a une valeur par défaut qui est de l'ordre de ~120%, sur des pages pleines de texte, genre mentions légales, ça fait très serré.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-scss-increase-default-line-height-of-text-cells-3643.patch 0001-scss-increase-default-line-height-of-text-cells-3643.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
Pour le line-height, je conseil toujours d'utiliser une valeur sans unité. Pour que la valeur lorsqu'elle est héritée se base comme référent sur le font-size de l'héritier.
line-height: 1.4;
<number> (unitless)
The used value is this unitless <number> multiplied by the element's own font size. The computed value is the same as the specified <number>. In most cases, this is the preferred way to set line-height and avoid unexpected results due to inheritance.
https://developer.mozilla.org/en-US/docs/Web/CSS/line-height
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
idéalement j'aurais eu tendance à fixer la valeur "default" du line-heigth dans un fichier _typography.scss
.
Ou dans notre cas dans _general.scss
.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Pour le line-height, je conseil toujours d'utiliser une valeur sans unité.
Oui pareil c'est pour ça que j'ai mis un pourcentage, qui pour moi fait exactement la même chose; la page côté MDN pointe un exemple qui n'utilise pas de %. Mais peu importe et ok pour faire sans.
idéalement j'aurais eu tendance à fixer la valeur "default" du line-heigth dans un fichier _typography.scss.
Ou dans notre cas dans _general.scss.
C'est de la prudence par rapport à tout l'existant, en posant ça uniquement pour les cellules texte je me rassure un peu.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-scss-increase-default-line-height-of-text-cells-3643.patch 0001-scss-increase-default-line-height-of-text-cells-3643.patch ajouté
Le patch modifié pour utiliser 1.4 plutôt que 140%, pour que ça s'applique aussi aux listes, et pour ne pas perdre les marges des <h3>.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
(Utopie: Ce serait bien quand même bien qu'on arrive à avoir un rythme vertical régulier et standard dans un thème : qu'un paragraphe quelque soit son emplacement (text-cell ou autre cell), ait le même line-height. Une vraie gestion des blancs typograhiques).
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 68681fe9eb6fbed23565eef746db867e6e3bfa36 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Sep 25 14:12:39 2019 +0200 scss: increase default line height of text cells (#36436)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: increase default line height of text cells (#36436)