spyrales / gouvdown

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

Thème ggplot2 #1

Closed tvroylandt closed 4 years ago

tvroylandt commented 4 years ago

Faire les éléments de thème pour ggplot2

MaelTheuliere commented 4 years ago

Ajouter un logo sur un ggplot : https://themockup.netlify.app/posts/2019-01-09-add-a-logo-to-your-plot/

tvroylandt commented 4 years ago

Pour le logo, il faudra distinguer deux cas :

On peut penser à une fonction wrapper comme la BBC avec finalise_plot() : https://bbc.github.io/rcookbook/#save_out_your_finished_chart

MaelTheuliere commented 4 years ago

On va être confronté à la gestion des palettes pour windows. Si j'ai bien suivi, il faudra pour ggplot un truc complémentaire à base d'extrafont pour importer les polices au chargement du package

tvroylandt commented 4 years ago

Ya ça qui peut aider : https://github.com/r-lib/systemfonts

MaelTheuliere commented 4 years ago

Super, merci, c'est le complément d'extrafont qui me manquait quand les polices ne sont pas installées dans le path par défaut.

tvroylandt commented 4 years ago

Je me demande si faudrait pas un geom_text avec une autre police par défaut aussi

RLesur commented 4 years ago

Lorsque les polices ne sont pas installées sur le poste, il me semble qu'on peut aussi utiliser showtext.

MaelTheuliere commented 4 years ago

J'ai un truc qui marche avec extrafont : 35d24a4

RLesur commented 4 years ago

C'est chouette mais êtes-vous sûr que tout le monde a le droit d'installer des polices sur sa machine ? Moi, par exemple, au boulot je ne peux pas... mais on peut espérer que ce sera installé sur tous les postes.

MaelTheuliere commented 4 years ago

Même sur ton espace user ? Je faisais l'hypothèse qu'il fallait aussi restreindre l'usage de la font à ceux qui peuvent l'installer...

MaelTheuliere commented 4 years ago

ça vous semble ok ? image

MaelTheuliere commented 4 years ago

obligé de mettre un fond un peu sombre pour la palette principale

tvroylandt commented 4 years ago

C'est super !

J'avoue que si on pouvait se passer du fond sombre (et donc de la palette FR de fait, ou au moins du blanc), je préfère.

MaelTheuliere commented 4 years ago

moi aussi ;-)

MaelTheuliere commented 4 years ago

on peut faire une version light qui serait celle par défaut, une version regular comme celle là

tvroylandt commented 4 years ago

De fait la palette par défaut dans ggplot2, c'est la A (la marron). Donc sur un fond blanc, elle passe. On pourrait mettre en défault la palette pal_gouv_qual1 qui reprend simplement tous les niveaux proposés et qui de fait va avec un thème blanc.

MaelTheuliere commented 4 years ago

Ca marche. Si je récapitule :

Et une modif de la palette par défaut là dessus pour être cohérent avec le theme light Si ça vous va je dois pouvoir boucler ça aujourd'hui.

MaelTheuliere commented 4 years ago

Une dernière question : j'ai ajouté le paramètre plot.title.position qui n'est disponible que dans la dernière version de ggplot2, ça demande à modifier les dépendances pour forcer les utilisateurs à mettre à jour. Je pense que ça vaut le coup, mais qu'en pensez vous ?

tvroylandt commented 4 years ago

Je suis d'accord avec tout.

Pour plot.title.position aussi.

J'allais même suggérer de passer en element_markdown()avec ggtext pour forcer aux bonnes pratiques de design de graph, mais ça rajoute une dépendance pour un truc un peu complexe en utilisation standard. On mettra plutôt ces bonnes pratiques dans le cookbook.

RLesur commented 4 years ago

Même sur ton espace user ?

Je n'ai pas poussé les essais très loin : je ne fais que tu HTML et j'ai décidé de tout déclarer en font face. Mais effectivement, je n'ai pas investi le sujet des graphiques.

Sur les autres sujets, je ne suis vraiment pas expert en graphique. Voici mon avis sur les différents points, pour ce que ça vaut :

MaelTheuliere commented 4 years ago

image

tvroylandt commented 4 years ago

C'est beau =)

RLesur commented 4 years ago

J'aime beaucoup également !

MaelTheuliere commented 4 years ago

J'ai pris un peu de retard, mais c'est bon. Ce sera au propre demain ;-)

image

RLesur commented 4 years ago

Vous avez vu ça ? Ça commence à peine mais ça me semble prometteur https://rstudio.github.io/thematic/

MaelTheuliere commented 4 years ago

oui j'ai vu passé ça ce matin dans ma veille Je me disais qu'il y avait un soucis pour les palettes qui sont définie de façon stricte en non par rapport à une couleur principale

MaelTheuliere commented 4 years ago

Au passage, j'ai un soucis avec gouvdown-imports.Rd qui me bloque pour l'utilisation du pipe dans les fonctions. J'ai l'habitude d'utiliser usethis::use_pipe() mais là évidemment les deux sont en conflits. Une idée ? Sachant que j'ai pas trouvé où était créé ce gouvdown-imports.Rd

tvroylandt commented 4 years ago

Oui, j'ai vu. C'est top. Après je connais pas grand monde qui utilise encore lattice et base pour les graphs. C'est bien quand on a de l'ancienneté et qu'on a commencé avec ça, mais tout le monde fait du ggplot2 maintenant et ceux qui apprennent maintenant apprennent comme ça. J'en faisais avant mais là je saurai même pas te faire un graph en base .

Après ça simplifie vachement la mise en place du thème pour le reste... Je vais peut-être l'utiliser pour ggdown, ça me simplifiera ce bout là.

RLesur commented 4 years ago

Au passage, j'ai un soucis avec gouvdown-imports.Rd qui me bloque pour l'utilisation du pipe dans les fonctions. J'ai l'habitude d'utiliser usethis::use_pipe() mais là évidemment les deux sont en conflits. Une idée ? Sachant que j'ai pas trouvé où était créé ce gouvdown-imports.Rd

Je pense que le mieux serait de discuter de ce sujet dans une pull request : je vais l'ouvrir en WIP.