Zenika-Training/zenika-formation-framework

Supporter plusieurs thèmes

Opened this issue · 10 comments

Dans le cadre des formations avec l'institut du développement du groupe La Poste, il faudrait pouvoir brander les formations avec leur chartes : Charte filière SI.pptx

cc @eigidi

Vous avez des idées d'implem ?

Bon, je commence à voire un peu comment c'est fait. Au niveau fonctionnel, pouvoir choisir un thème au build (via une propriété dans le package.json ou comme paramètre de commande de build de la formation qui utilise le framework) serai idéal. Maintenant, pour l'implem, il faut que je me familiarise avec le framework pour voir comment faire ça au mieux (suggestion bienvenue ;)

Ah non, mais ok, j'ai compris comment marche le truc. En fait, y'a rien à faire ici dans la mesure ou il y à déja ce qu'il faut : Slides/ressources/custom.css

@zigarn Je te prépare une feuille de style au petit oignons pour ta formation et je te prépare une PR pour https://github.com/Zenika/formation-devops-mindset-tooling

Bon, après, qu'on ai un mécanisme un peu générique pour gérer les styles ce ne serais pas déconnant dans l'absolue. Mais je dirais qu'un peu de ménage dans l'outil serai un peu un pré-requis pour ça (cf PR #228 )

Parfait je te prépare ça :)

Peut-on fermer cette issue?

Non.
Le choix de thème devrait être porté par le framework.
(Cela ne signifie pas forcément que le thème soit dans le framework lui-même).

Actuellement, sur la formation ayant besoin du thème La Poste, on est obligé de gérer le thème dans une branche car si on la merge la formation n'aura plus que ce thème, même lorsque l'on veut la donner en dehors de La Poste.

je met une pièce dans la demande, aujourd'hui formation avec une personne malvoyante, besoin de faire un jeu de slide texte blanc, fond noir ..... a l’écran et a l'impression .... du coup quelques soucis avec l’empilement des css ...

Le framework actuel supporte 2 thèmes et on peut changer le thème avec la touche T. Cependant le fait d'avoir 2 thèmes est codé en dur ici et ici. Une PR pour supporter une liste arbitraire de thèmes (embarqués, pas externes) est la bienvenue. Une PR qui ajoute ensuite un thème embarqué pour personnes mal voyantes est aussi la bienvenue. Enfin une PR pour supporter les thèmes externes (c'est-à-dire pas commité dans ce repo) est également envisageable.