[spip-dev] Compression CSS : erreur d'analyse

Dev
1

Salut les spipeurs,

J'ai activé la compression CSS de Spip et on vient de me faire savoir
que la console de Firefox remontait plein d'erreurs.

Je vais donc regarder, et voilà un des points en erreur :
`h1{font:700 200% inherit}`
… qui correspond dans ma CSS avant compression à :
`h1 { font-size:200%; font-family:inherit; font-weight:bold; }`

Firefox dit : « Erreur d'analyse de la valeur pour « font ».
Déclaration abandonnée. »

Est-ce que c'est connu et déjà patché et ça arrivera dans la 3.1 par
exemple, ou est-ce que vous voulez une analyse plus précise et une
suggestion de correction ?

2

Hello,

Je crois pas que ce cas ait été remonté. Le minifieur condense en effet les 3 règles en une seule règle font, mais on dirait que
font:700 200% inherit
n'est pas acceptée avec un inherit je suppose ?

Le navigateur attend une font family explicite, et pas un inherit ?

3

Bonjour Cédric.

Je crois pas que ce cas ait été remonté.

Et pourtant, j'ai déjà signalé ce problème depuis plus d'un an maintenant:

Didier

4

Il se peut effectivement que ce soit ça, cf.
https://drafts.csswg.org/css-fonts-3/#propdef-font-family :
Name: font-family
Value: [ <family-name> | <generic-family> ] #

Auquel fait référence https://drafts.csswg.org/css-fonts-3/#font-prop :
Name: font
Value: [ [ <‘font-style’> || <font-variant-css21> || <‘font-weight’> ||
<‘font-stretch’> ]? <‘font-size’> [ / <‘line-height’> ]? <‘font-family’>
] | caption | icon | menu | message-box | small-caption | status-bar

Je n'ai pas tout lu mais on dirait bien qu'effectivement inherit n'a pas
sa place ici.

5

Ah mais oui, toutes mes excuses, je suis passé à côté de cette PR en faisant mon lot d'intégration en octobre l'an dernier.

Le bug est donc connu, identifé, et il y a un patch proposé.
Il faut juste que je merge à la lib et que j'ajoute un cas dans les tests unitaires pour couvrir ce point, puis ensuite que je mette à jour la lib dans le plugin compresseur de SPIP !

Je m'en occupe rapidement.

6

C'est intégré dans les branches 3.0 et 3.1, ce sera donc dans la prochaine mise à jour de SPIP :slight_smile: