Zenika-Training / zenika-formation-framework

Tools to build and present our training material
8 stars 25 forks source link

feat(#171): replace highlight.js by prism.js #172

Closed Swiip closed 7 years ago

Swiip commented 7 years ago

Pas sur que ce soit complet mais ça a l'air de marchouiller déjà.

De plus, cela pose une question de lister les langages supporté par défaut pour le framework. Pour l'instant, je vois pas bien comment une formation pourra en ajouter un qui n'est pas déjà prévu.

Fix #171

Swiip commented 7 years ago

Yop, merci pour les retours je m'en charge dès que je peux.

Pour les commentaires, je fais les mêmes retours d'habitude mais là j'avais des remords :p Pour le CSS, je crois que j'ai déjà un peu regardé sans succès mais je vais creuser un peu

Swiip commented 7 years ago

Voilà pour les retours.

Je suis inquiet par contre pour le script de génération automatique du cahier d'exercice qui utilise encore highlight il semblerait.

hgwood commented 7 years ago

Pour la génération du cahier d'exercice je pense que tu as juste à changer le chemin ici : https://github.com/Zenika/zenika-formation-framework/blob/master/Gruntfile.js#L297

Swiip commented 7 years ago

J'ai un peu regardé, c'est plus compliqué que ça. Cette ligne correspond à la config du CSS pour highlight.js mais highlight.js lui même n'est pas optionnel mais en dur dans la librairie markdown-pdf.

Je ne vois pas comment mettre prism dans le cahier d'exercice. Du moins proprement, j'imagine bien un hack dégeulasse, mais j'ose même pas en parler ^^

hgwood commented 7 years ago

OK. Est-ce que la génération du cahier fonctionne toujours correctement avec highlight.js quand même ? On peut merger ?

Swiip commented 7 years ago

J'ai fais un passage mais je n'arrive pas à faire marcher la génération du cahier chez moi. J'ai le droit à un joli Fatal error: Command failed: ...

Mais à priori, le seul truc qui avait changé, c'est que j'avais viré code.css. Je l'ai remis, du coup ça doit être iso.

hgwood commented 7 years ago

On dirait que tu n'es pas sûr. As-tu testé la génération du cahier d'exercice avec le code.css remis à sa place ?

Swiip commented 7 years ago

Non c'est ce que je dis, je n'arrive pas à le faire marcher sur mon poste. Ni avant ni après.

Swiip commented 7 years ago

🎉