#4861 : Simplifier le code W3C / conformité HTML5
Modifications en conformité avec le w3c, notamment les balises n'ont pas de end tags, voir https://html.spec.whatwg.org/multipage/syntax.html#syntax-end-tag
Rapports de requête de fusion
Activité
@touti a ajouté 8 révisions : 11fcfa80b0b4b186c7c2233fde8c1ef8b33f9f1b df8fe208adcd1bc67e6b327dff5c2394df5f6623 cd2017cdbd905eaba045b5bfca4167f0f8221463 d2ffcd10946d0bfc531c0942623af1c57bdfbe22 9a37ca20ef0d971824f6cc1637c093f93b87a878 2f10b26ac7cacdf926df0546ce4576e04e7cdfec 28899508876f293c52f422c6ec3da57fa42c2b69 c703edbb20d64e4a0151d6e45916f568a41106db
Alors quelques petites remarques sur cette PR relative à #4861 (closed)
Merci tout plein !
- Ça serait bien d’utiliser pour les commits le formalisme des «conventional commit» : https://www.conventionalcommits.org/en/v1.0.0/
- Et donc d’indiquer des
fix:
(oufeat:
) le cas échéant, et une ligneRefs: #4861
référant au ticket concerné - De squasher certains commit pour éviter des
oups
ou des commits doublés - De faire un commit type
docs(changelog):
pour le CHANGELOG.md associé
Autre remarque alternative : je me demande si on peut pas enlever tous les
/*<![CDATA[*/
et/*]]>*/
aussi, non ? qui me semblent dater d’une autre époque ?Dis nous si tu ne te sens pas pour altérer / rebaser / renommer les commits de ta PR.
@marcimat a remplacé le titre
fix issue4861par #4861 (closed) : Simplifier le code W3C / conformité HTML5Autre remarque alternative : je me demande si on peut pas enlever tous les // aussi, non ? qui me semblent dater d’une autre époque ?
+1 Ça semble être un reliquat du temps ou la dist était en XHTML, ça n'est à priori plus nécessaire non ?
https://stackoverflow.com/questions/66837/when-is-a-cdata-section-necessary-within-a-script-tag
Autre remarque alternative : je me demande si on peut pas enlever tous les // aussi, non ? qui me semblent dater d’une autre époque ?
+1 Ça semble être un reliquat du temps ou la dist était en XHTML, ça n'est à priori plus nécessaire non ?
Oui oui, c’est ce que j’en avais compris également, à quelques exceptions chargeant des trucs externes près (svg, mathml) de mémoire
D'ailleurs dans le lien mentionné par @touti il y a un passage sur CDATA https://html.spec.whatwg.org/multipage/syntax.html#cdata-sections
"CDATA sections can only be used in foreign content (MathML or SVG)"
Bonjour,
Dis nous si tu ne te sens pas pour altérer / rebaser / renommer les commits de ta PR.
Oui, exactement :) Et promis, j'essayerai de suivre https://www.conventionalcommits.org/en/v1.0.0/ mais je ne suis qu'une petite chose isolée, un soupçon de pet galactic, qui code dans son garage, je ne demande qu'à apprendre. J'ai déjà évité de mettre sur la forge les fichiers enregistrés en #sauvegarde en faisant un git reset --hard 3u38t0qn00p1sr1o8012r16kp2nsxcmul1w426w5 :p
@marcimat a force-pushed 2 révisions : c703edbb20d64e4a0151d6e45916f568a41106db 00c563da06458e852eef72126c57889d81961a0e
J’ai rebasé / fixup et amandé les commits.
Cela dit, j’ai un doute sur certains changements, dans
- backend*.html tel que ce backend
- rss_forum_*.html tel que ce rss_forum…
Ça renvoie du XML, du coup il faut conserver les fermetures non ?
@b_b a approuvé cette PR (00c563da06458e852eef72126c57889d81961a0e)
Sur les
<br>
j'ai un doute https://developer.mozilla.org/en-US/docs/Web/HTML/Element/brTag omission Must have a start tag, and must not have an end tag. In XHTML documents, write this element as
.Sur les
<br>
j'ai un doute https://developer.mozilla.org/en-US/docs/Web/HTML/Element/brTag omission Must have a start tag, and must not have an end tag. In XHTML documents, write this element as
.Non non, le changement est valide, on est bien en html5 maintenant.
@b_b a demandé une évaluation