-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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.
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
Fix #14447 #15884
Fix #14447 #15884
Conversation
Preview URLs Flaws (2)URL:
External URLs (1)URL:
(comment last updated: 2023-09-25 10:30:00) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok for the content 👌🏻
{{EmbedLiveSample('Exemples', '230', '420')}} | ||
#### Résultat | ||
|
||
{{EmbedLiveSample('Définir la colonne de début pour un élément de grille', '230', '420')}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SphinxKnight Par pur suivi des méthodes US, est-ce qu'il ne serait pas mieux de mettre les underscores à la place des espaces sur la valeur 1 (c'est vraiment par pure convention personnelle sur les nommages que j'ai cette question, si tu n'en vois pas l'utilité je serai pas contre ton point de vue ^^)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pour expliciter la démarche : j'ai enlevé l'argument et ça ne fonctionnait pas (sans doute un cas de mdn/yari#5005 ), j'ai eu un souvenir fugace qu'une "amélioration" avait été apportée pour pouvoir utiliser le titre tel quel (plutôt que slugifié) et donc voilà.
Bref, j'ai eu la flemme et tu as raison :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pas de soucis xD
|
||
#### Résultat | ||
|
||
{{EmbedLiveSample('Placement relatif aux lignes', '230', '220')}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voir question relative plus haut.
An update to fix #14447 + an addition of a new embedded example.