Project

General

Profile

Development #36436

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

Added by Frédéric Péters 10 months ago. Updated 4 months ago.

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

Associated revisions

Revision 68681fe9 (diff)
Added by Frédéric Péters 4 months ago

scss: increase default line height of text cells (#36436)

History

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

#2 Updated by Thomas Jund 10 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 10 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 10 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 4 months 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 4 months 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).

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

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

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

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF