New theme!

[hgwood]

Closes #214. Screenshots incoming.

todo

• [x] tableaux plus léger
• [x] éloigner un peu les titres des slides des triangles rouges
• [x] aligner le titre des slides avec le contenu
• ajouter les triangles sur les côtés des titres de chapitre (décalé, issue à ouvrir éventuellement)
• [x] tester le rendu PDF
• [ ] corriger le fait que les slides de plan de la formation Node.js restent blanches (parce que le contenu dépasse de la slide)
• [x] bug : double bordures latérales

[hgwood]

[fabienjuif]

Ca rend vraiment bien ;)

Il y a moyen d'ajouter une indentation pour les snippets de code ? Seulement quand c'est dans une liste à puce (je ne pense pas que ça ça soit possible) :)

[hgwood]

@fabienjuif en fait si ton code black est collé à ton bullet point en markdown alors le code block fera partie de la balise li et sera correctement indenté. Par contre, l'indentation du code à l'intérieur du bloc reste à absolue donc on se retrouve avec du code non indenté.

Exemple

- Par convention dans Node.js
  - Le premier argument correspond à une éventuelle erreur
  - Les autres arguments sont les données de l'évènement

``javascript
let callback = function(err, result) {
  if (err) {
    console.error(err.message);
  } else {
    console.log(result)
  }
};
``

- Par convention dans Node.js
  - Le premier argument correspond à une éventuelle erreur
  - Les autres arguments sont les données de l'évènement
``javascript
let callback = function(err, result) {
  if (err) {
    console.error(err.message);
  } else {
    console.log(result)
  }
};
``

Le problème n'est donc pas dans la conversion markdown->html ni dans le css du thème mais plutôt dans le plugin qui highlight le code.

[olivierhuber]

C'est top ! Je vois qu'il reste 3 todo donc on attend avant de merger ?

[hgwood]

Oui encore deux bugs importants.

[olivierhuber]

Plus que 1 issue ... Bravo ! Pensez vous que nos formations de la semaine prochaine pourrons en bénéficier ?

[hgwood]

@olivierhuber Not available myself until December 1st.

[hgwood]

I'm getting stuck on that last problem. But it is really important that it's fixed before applying the new theme. It's better to have the content properly printed in the old theme than to have it truncated in the new theme.

[hgwood]

Screenshots of the problem

Everything works fine when viewing the slides in the browser, even if print media is emulated.

But in the printing preview and the final PDF, the content is gone

[hgwood]

Works fine in Firefox. Might be a Chrome bug.

[hgwood]

Both Chrome stable 62.0.3202.94 and canary 64.0.3282.5 on Windows 10 reproduce the problem.

[hgwood]

Bug reproduced on Opera on macOS too.

[hgwood]

I might merge this but keep it behind some kind of flag or keyboard shortcut. The best would be to have the new theme for the HTML format (it works fine there) but leave the old theme for PDF generation. I'll try to do that.

[hgwood]

I'm still planning to do this! I should check if that bug is still present.

[bpetetot]

I made some tests on the ReactJS formation about the PDF rendering bug.

Some facts that I can't really explain :

• The agenda with an overflow works correctly
• On some slides with overflow, I have blank PDF rendering, and I could resolve them by changing the markdown content (but still in overflow) :

before (blank slide):

after - adding blank lines between code (slide ok) :

[hgwood]

Dans la version que je viens de pousser, les deux thèmes sont dispos. L'ancien thème est par défaut, pour que les PDF se génèrent bien. On peut appuyer sur la touche 't' pour changer de thème.

Quand il sera temps de supprimer l'ancien thème, il faudra :

• supprimer reveal/theme-zenika/pdf.css
• supprimer reveal/theme-zenika/theme.css
• supprimer reveal/theme-zenika/images
• supprimer reveal/theme-zenika/fonts
• modifier index.html et modifier link#theme pour que href pointe vers reveal/theme-zenika/theme-2017.css
• modifier reveal/theme-zenika/run.js
  • dans la fonction applyPrintStylesheets, supprimer le bloc if (window.location.search.match(/theme-2017/gi))
  • supprimer la fonction cycleThemes
  • supprimer le binding à la touhe 't' (key code 84)

[hgwood]

@bpetetot pendant que tu es dans le bain tu voudrais bien me tester ça ?