search

GouvernementFR / dsfr

🇫🇷 Official french government's design system (Système de Design de l'État)
https://www.systeme-de-design.gouv.fr/
Other
243 stars 54 forks source link

Fichier généré invalide #593

Closed l-vincent-l closed 1 year ago

l-vincent-l commented 1 year ago

Décrire le bug

Le fichier CSS distribué ne respecte pas les standards CSS3.

Les étapes pour reproduire le bug

Si on passe le fichier https://cdn.jsdelivr.net/npm/@gouvfr/dsfr@1.9.2/dist/dsfr.css dans https://jigsaw.w3.org/css-validator/ on obtient un certain nombre d’erreurs.

Le rapport est visible ici

On note une première erreur simple à rectifier no-wrap -> nowrap Des erreurs autour des valeurs de aspect-ratio non conforme, ce que je n’explique pas. C’est en fait mal géré par le validateur Des paddings négatifs ce qui est contraire à la spécification CSS. Et finalement des erreurs de syntaxe. C’est en fait mal géré par le validateur

Comportement attendu

On ne voudrait pas qu’il y ait d’erreurs.

Capture d’écran

Configuration et système utilisé

Informations complémentaires

l-vincent-l commented 1 year ago

Pour les ratios, je pense que l’erreur vient du validateur voir l’issue https://github.com/validator/validator/issues/1406

l-vincent-l commented 1 year ago

Je pense que les erreurs de syntaxe sont causé par l’utilisation de selector qui est encore en expérimental, et pas implémenté par le validateur.

C’est confirmé ici https://github.com/w3c/css-validator/issues/361

En vraies erreurs il reste donc les padding négatifs, et en tant que développeur back, je ne me sens pas du tout de les corriger sans tout casser :grinning:

lab9fr commented 1 year ago

Merci pour ce rapport de bug

Je créé un ticket concernant les padding négatifs. Sans que ce soit dommageable, ça prend du poids inutilement.

Bertrand pour l'équipe DSFR

lab9fr commented 1 year ago

corrigé en v1.10