spyrales / gouvdown

French government design system for R Markdown
https://spyrales.github.io/gouvdown/
European Union Public License 1.2
48 stars 4 forks source link

Ajout d'un template bookdown #26

Closed MaelTheuliere closed 3 years ago

MaelTheuliere commented 3 years ago

Ajout d'un template bookdown marque Etat

MaelTheuliere commented 3 years ago

Qu'en pensez vous ?

Capture d’écran de 2020-09-18 11-19-16

Alternative : mettre le logo en dessous du sommaire... Capture d’écran de 2020-09-18 11-26-20

J'ai aussi une version fond bleu mais plus compliqué pour le bloque marque :

Capture d’écran de 2020-09-18 11-23-11

tvroylandt commented 3 years ago

Top ! Je préféère le logo fond blanc en haut

MaelTheuliere commented 3 years ago

@tvroylandt @RLesur dernier point sur gitbook gouv, J'ai besoin de plusieurs fonctions internes de bookdown. On est d'accord que dans ce cas je les recréé dans gouvdown ?

tvroylandt commented 3 years ago

Oui

MaelTheuliere commented 3 years ago

ça veut dire par exemple rajouter aussi tout ça dans gouvdown :

MaelTheuliere commented 3 years ago

Et tracer les modifs faites ensuite sur bookdown pour être à jour ?

tvroylandt commented 3 years ago

On a pas tellement le choix de toute façon non ? Puisque si c'est de l'internal c'est pas exporté ?

Ou alors il y a un moyen que je ne connais pas

MaelTheuliere commented 3 years ago

En fait l'idéal serait que bookdown intègre une paramétrisation dans gitbook sur la sortie utilisée => https://github.com/rstudio/bookdown/blob/master/R/gitbook.R#L29 On est obligé de tout dupliquer pour pouvoir remplacer à cet endroit html_document() par html_gouv(). T'en penses quoi toi @RLesur ?

MaelTheuliere commented 3 years ago

honnêtement, ça demande quasiment à réimporter la moitiée de bookdown, je vous propose donc de faire une entorse aux bonnes pratiques en appelant les fonctions non exportées de bookdown pour avancer et de travailler à rendre bookdown plus paramétrable. Je vois deux possibilités de hack sur le code ici : https://github.com/rstudio/bookdown/blob/master/R/gitbook.R#L29

Ca me semble jouable de faire accepter cela par les auteurs du package ;-)