search

Zenika-Training / zenika-formation-framework

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

Supporter plusieurs thèmes #242

Open zigarn opened 5 years ago

zigarn commented 5 years ago

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

hgwood commented 5 years ago

Vous avez des idées d'implem ?

JeremiePat commented 5 years ago

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

JeremiePat commented 5 years ago

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

JeremiePat commented 5 years ago

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 )

zigarn commented 5 years ago

C'est pour la formation https://github.com/Zenika/formation-devops-seminaire

JeremiePat commented 5 years ago

Parfait je te prépare ça :)

hgwood commented 5 years ago

Peut-on fermer cette issue?

zigarn commented 5 years ago

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.

LaurentTardif commented 4 years ago

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

hgwood commented 4 years ago

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.