Project

General

Profile

Development #36436

divers, augmenter le line-height par défaut des cellules texte

Added by Frédéric Péters 7 months ago. Updated 6 days ago.

Status:
Solution validée
Priority:
Normal
Target version:
-
Start date:
25 Sep 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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

0001-scss-increase-default-line-height-of-text-cells-3643.patch View (984 Bytes) Frédéric Péters, 25 Sep 2019 02:15 PM

0001-scss-increase-default-line-height-of-text-cells-3643.patch View (1011 Bytes) Frédéric Péters, 31 Mar 2020 02:26 PM

History

#1 Updated by Frédéric Péters 7 months ago

#2 Updated by Thomas Jund 7 months ago

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

#3 Updated by Thomas Jund 7 months ago

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.

#4 Updated by Frédéric Péters 7 months ago

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.

#5 Updated by Frédéric Péters 9 days ago

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

#6 Updated by Thomas Jund 6 days ago

  • Status changed from Solution proposée to 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).

Also available in: Atom PDF