Open lucsorel opened 6 years ago
Qu'est-ce qu'il reste à faire dans cette PR ?
Bonjour Patrice,
Il reste des cases à cocher dans la todo-list du README.md. Mais c'est à compléter. L'état aujourd'hui :
<img src="path/to/img.png">
) plutôt que via la syntaxe markdown (avec l'expression ![](path/to/img.png)
). Soit on arrive à faire passer les fichiers markdown dans le loader html classique pour qu'il repère les appels à des images sans que ça casse (je ne pense pas que le loader html apprécie le code markdown, surtout s'il fonctionne avec un parseur ou un arbre syntaxique de code), soit il faut faire évoluer le loader markdown-image-loader et c'est beaucoup plus chaud. En effet, ce loader utilise quelques regexp pour trouver et traiter les inclusions d'image, mais si je commence à mettre des regexp pour détecter du code html, alors on sait que je suis mettre le doigt dans des engrenages infernaux (il a toutefois une suite de tests unitaires existants, ça serait facile d'en rajouter par rapport à ces cas)Ton aide est la bienvenue si tu le souhaites.
Je tente une approche différente ici : https://github.com/Zenika/sensei
En gros au lieu d'essayer d'extraire les images du markdown, je transforme le markdown en html et j'utilise le html-loader pour extraire les images. Avantage : le rendu du markdown se fait au build au lieu du runtime, et je suis sûr de choper toutes les images. Inconvénient : j'ai du écrire un loader qui reproduit le pré-traitement que Reveal.js fait sur le markdown (split d'un fichier en slides, extraction des notes, extraction des annotations <!-- .slide: page-demo -->
).
Qu'est-ce qu'il reste à faire dans cette PR ?