Closed MaelTheuliere closed 3 years ago
Merging #30 into master will decrease coverage by
1.08%
. The diff coverage is50.00%
.
@@ Coverage Diff @@
## master #30 +/- ##
==========================================
- Coverage 67.63% 66.55% -1.09%
==========================================
Files 7 7
Lines 275 287 +12
==========================================
+ Hits 186 191 +5
- Misses 89 96 +7
Impacted Files | Coverage Δ | |
---|---|---|
R/html_gouv.R | 76.19% <50.00%> (-13.81%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update ae1f17e...5a332ac. Read the comment docs.
Est ce que vous avez commencé à regarder @tvroylandt @RLesur ? Sinon je me propose un droit au remord sur html_gouv()
. On aura besoin d'une version sans logo pour bookdown...
Dites moi et je rajoute un commit si c'est bon pour vous
Au passage j'ai presque bouclé le gitbook_gouv()
, il manque juste la gestion du logo dans le template de projet...
Cette PR permet de gérer les différents problèmes listés sur l'issue #24 apparus avec la PR #21 Sur la gestion du header, je propose d'internaliser la création dans
html_gouv()
mais de garder quand même le fichier résultat (header.html
) dans le répertoirewww
si les utilisateurs souhaitent le customiser ensuite. Attention @tvroylandt à prendre en compte ensuite dans tes travaux sur le template paged. J'en ai profité pour rajouter le bloc-marque "République Française" qui me semble plus pertinent d'avoir par défaut. J'ai testé les différentes configurations sur les polices, il semble bien prendre en compte ou gouvdown.fonts ou les polices en local. Mais je suis preneur de vos retours ;-)