Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

WIP: webpackify the framework #228

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

WIP: webpackify the framework #228

wants to merge 4 commits into from

Conversation

lucsorel
Copy link

@lucsorel lucsorel commented Feb 2, 2018

No description provided.

@patou
Copy link
Contributor

patou commented Oct 3, 2019

Qu'est-ce qu'il reste à faire dans cette PR ?

@lucsorel
Copy link
Author

lucsorel commented Oct 3, 2019

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 :

  • je réalise avec horreur que ça fait 2 ans que je n'ai pas pu avancer là-dessus :/ ; il faudrait aussi passer à webpack4 :p . Le projet utilise un loader que j'ai développé (https://github.com/lucsorel/markdown-image-loader) pour gérer les inclusions d'image dans les fichiers markdown. Il est compatible webpack (les exemples en ligne sont pour webpack 4)
  • la branche est en retard de 10 commits de master, et le rebase peut faire mal
  • le markdown des formations inclut parfois les images via du code html natif (avec une balise <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.

@hgwood
Copy link
Contributor

hgwood commented Oct 14, 2019

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 -->).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants