Support des backticks pour insérer du code #4843 #4849

Merged
cerdic merged 4 commits from issue_4843 into master 2 months ago
cerdic commented 3 months ago
Owner
  • support des backticks simples, doubles et triples comme markdown. Le html produit est le même que pour les balises <code> a la seule différence qu'il est trimé (ce que fait markdown)
  • jeu de tests pour le support des backticks
* support des backticks simples, doubles et triples comme markdown. Le html produit est le même que pour les balises `<code>` a la seule différence qu'il est trimé (ce que fait markdown) * jeu de tests pour le support des backticks
cerdic added 3 commits 3 months ago
b_b approved these changes 3 months ago
Owner

a la seule différence qu'il est trimé

Ça sucre donc les sauts de lignes vides en fin de bloc de code à trois `, c'est bien ça si j'ai bien lu les tests ? => testé, ça trim avec cet exemple :

<?php

hop





Ça m'étonne, car ici et sur discuter.spip ça n'est pas le cas (ça l'est uniquement pour les simples `).

Et je viens de tester avec la preview markdown dans codium, ça ne trim pas sur les blocs à base de trois `. Idem ici https://markdownlivepreview.com/ et là https://dillinger.io/

> a la seule différence qu'il est trimé Ça sucre donc les sauts de lignes vides en fin de bloc de code à trois `, c'est bien ça si j'ai bien lu les tests ? => testé, ça trim avec cet exemple : ``` <?php hop ``` Ça m'étonne, car ici et sur discuter.spip ça n'est pas le cas (ça l'est uniquement pour les simples `). Et je viens de tester avec la preview markdown dans codium, ça ne trim pas sur les blocs à base de trois `. Idem ici https://markdownlivepreview.com/ et là https://dillinger.io/
Poster
Owner

Ah oui je me suis peut-être emballé. J'allais dire "j'ai repris la spec de parsedown" mais justement non, pas sur les ``` :p

Donc en effet on doit pouvoir ne pas trimer sur les blocs car du coup ça peut-être utile de laisses des lignes vides à dessein

Ah oui je me suis peut-être emballé. J'allais dire "j'ai repris la spec de parsedown" mais justement non, pas sur les ` ``` ` :p Donc en effet on doit pouvoir ne pas trimer sur les blocs car du coup ça peut-être utile de laisses des lignes vides à dessein
Owner

Quitte à faire ça, il faudrait générer un <pre> pour les blocs de code pour l’encapsuler, et reporter aussi le language de programmation des triples ticks sur la balise code ?

Voir le plugin precode https://contrib.spip.net/Pre-Code

et aussi les tickets

https://prismjs.com/ aussi qui mets class=language-xxx sur la balise code

Quitte à faire ça, il faudrait générer un `<pre>` pour les blocs de code pour l’encapsuler, et reporter aussi le language de programmation des triples ticks sur la balise `code` ? Voir le plugin `precode` https://contrib.spip.net/Pre-Code et aussi les tickets - https://git.spip.net/spip/spip/issues/3990 - et avant https://git.spip.net/spip/spip/issues/2504 https://prismjs.com/ aussi qui mets `class=language-xxx` sur la balise `code`
cerdic added 3 commits 2 months ago
cerdic added 1 commit 2 months ago
Poster
Owner

J'ai modifié pour générer le html via la fonction du core spip_balisage_code() introduite par spip/spip#5351

J'ai modifié pour générer le html via la fonction du core `spip_balisage_code()` introduite par https://git.spip.net/spip/spip/pulls/5351
cerdic merged commit 15eeed72a6 into master 2 months ago
cerdic deleted branch issue_4843 2 months ago

Reviewers

b_b approved these changes 3 months ago
The pull request has been merged as 15eeed72a6.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.