Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fichier généré invalide #593

Closed
l-vincent-l opened this issue Apr 26, 2023 · 4 comments
Closed

Fichier généré invalide #593

l-vincent-l opened this issue Apr 26, 2023 · 4 comments
Assignees
@lab9fr

Comments

@l-vincent-l
Copy link
Contributor

l-vincent-l commented Apr 26, 2023 •
edited
Loading

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é

  • **Version du DSFR : ** 1.9.1
  • **Appareil (mobile, tablette, desktop) : ** validateur CSS

Informations complémentaires
@l-vincent-l
Copy link
Contributor Author

Pour les ratios, je pense que l’erreur vient du validateur voir l’issue validator/validator#1406

@l-vincent-l l-vincent-l mentioned this issue Apr 26, 2023
Merged
@l-vincent-l
Copy link
Contributor Author

l-vincent-l commented Apr 26, 2023 •
edited
Loading

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 w3c/css-validator#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 😀

@lab9fr
Copy link
Contributor

lab9fr commented May 17, 2023

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 lab9fr self-assigned this May 17, 2023
lab9fr pushed a commit that referenced this issue May 17, 2023
@l-vincent-l
🐛 fix(core): La valeur no-wrap n’existe pas dans la spécification CSS (…
2c69052 
…#594)

Il y a un `-` en trop.

Voir https://www.w3.org/TR/css-flexbox-1/#flex-wrap-property

partial-fix de #593
@lab9fr
Copy link
Contributor

lab9fr commented Sep 8, 2023

corrigé en v1.10

@lab9fr lab9fr closed this as completed Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lab9fr lab9fr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lab9fr @l-vincent-l