Projet

Général

Profil

Bug #68796

La définition d’une variable $font-line-height dans le _vars.scss du thème reste sans effet (?)

Ajouté par Paul Marillonnet il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 septembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Agate et moi constatons ce même comportement.
Par exemple le thème Plessis-Trévise où je tente un

$font-line-height: 1.5;

Et c’est toujours la valeur 1.4 attrapée depuis static/includes/_general.scss qui est appliquée.

Est-ce qu’on loupe un truc ?


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #68742: MNHN : Intégration graphique initialeFermé06 septembre 2022

Actions

Révisions associées

Révision 51f4a0e2 (diff)
Ajouté par Paul Marillonnet il y a plus d'un an

general: allow overloading of font line height default variable (#68796)

Historique

#1

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Description mis à jour (diff)
#2

Mis à jour par Paul Marillonnet il y a plus d'un an

#3

Mis à jour par Serghei Mihai il y a plus d'un an

Il manque !default dans la déclaration dans _general.scss pour permettre la surcharge de la valeur.

#4

Mis à jour par Paul Marillonnet il y a plus d'un an

Bien vu. Testé sur le thème Plessis-Trévise, c’est bon pour moi.

#5

Mis à jour par Serghei Mihai il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Ok, pousser demain pour ne pas faire la même bêtise que moi: #68612

#7

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 51f4a0e2aeb81db5cc0d1a83bf51cb83ffc1bc4e
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Sep 8 11:02:04 2022 +0200

    general: allow overloading of font line height default variable (#68796)

#8

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF