Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mise à jour du template rmarkdown #30

Closed
wants to merge 3 commits into from
Closed

Conversation

MaelTheuliere
Copy link
Collaborator

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épertoire www 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 ;-)

@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #30 into master will decrease coverage by 1.08%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            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%) ⬇️

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.

@MaelTheuliere
Copy link
Collaborator Author

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

@MaelTheuliere MaelTheuliere mentioned this pull request Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants