Project

General

Profile

Development #36436

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

Added by Frédéric Péters about 2 months ago. Updated about 2 months ago.

Status:
Solution proposé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

History

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

#2 Updated by Thomas Jund about 2 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 about 2 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 about 2 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.

Also available in: Atom PDF