# Interrogations sur la MR 165 : *issues 137 et 175 modifs sur les illustrations*

Issues:
https://gitlab.com/rdes_dreal/propre.rpls/-/issues/137
https://gitlab.com/rdes_dreal/propre.rpls/-/issues/175

## Est-ce qu'on pourrait rendre paramétrable des notes de lecture pour accompagner le lecteur ? ou est-ce qu'on peut modifier le footnote dans la fonction avant ?

AW a pour cela créer un argument `note_de_lecture` dans chaque fonction de carte, graphe, tableau: **OK pour moi**
> ok aussi sur le principe, j'ai pas vérifié si la footnote qui préexistait a bien été conservé, c'est important que ce qu'on propose par défaut reste identique aux versions précédentes

> *OUI il l'a rajouté un plus de l'existant* :+1:

## Proposer un titre informatif et un titre technique par illustration

AW a mis un nouvel argument `titre_informatif` dans chaque fonction de carte, graphe, tableau:
- pour les cartes et les graphes, cela correspond aux sous-titre (sauf pour le graphe 1.1)
- pour les tableaux et le graphe 1.1, c'est dans le retour à la ligne du caption/Titre

Il a ensuite créer une argument `voulez_vous_le_titre_technique` (booleen) uniquement dans les fonctions des tableaux et le graphe 1.1 qui permet d'enlever ou non le titre par défaut.

**Je m'interroge sur:**
- le fait de ne pas retrouver l'argument dans toutes les fonctions d'illustrations. Je serais tentée de l'enlever de graphe 1.1 pour trouver plus tard une solution valable pour tous les graphes.
> pourquoi ne pas faire aux autres fonctions de création de graphes ce qui a été fait pour 1_1 ?

> Arnaud:
>j'ai mis la possibilité d'enlever les titres techniques auto uniquement au tableau et au 1er graph
pour les autres graphs on peut pas mettre plus de 2 lignes à un titre donc ça ferait moche juste un titre informatif en petit comme les unités
TODO pour l'année prochaine si on a un retour sinon flemme car faut un peu plus jongler pour graph et cartes :)

> Je vais tester, je ne comprends pas 'on peut pas mettre plus de 2 lignes à un titre pour les autres graphiques'

- l'intérêt de pouvoir enlever le titre par défaut.
> --> ça peut servir si le titre informatif est redondant avec le titre par défaut, par exmeple "Une évolution du nombre de logements sociaux de 15 % en 5 ans"
> OK!
- et peut être renommer la variable plus simplement: titre_par_defaut: TRUE/FALSE :ok:

> **COMPLETER NEWS.Rd**
> Et surtout le guide de prise en main au niveau du parametrage des illustrations

## Si c'est possible d'enlever la titraille automatique table1 dans le tableau 1, car les graphes et cartes n'ont pas de titre auto ?

- [x] **OK** grâce à l'aide sur GRRR: commit fait

## Ajouter la possibilité de plier/déplier les types d'EPCI (Métropole/CA/CC) ou piste : utiliser kableExtra::scrollbox() pour les grosses tables html pour que ce soit plus lisible quand on met les EPCI dedans ?

Ok, par contre, le scroll vertical rajoute de la largeur au tableau et du coup impose un scroll horizontal.... (cf capture ci dessous)
Le scroll est appliqué qu'on choisisse les EPCI ou non.
> faut faire tester aux éditos, mais pour moi le scroll ne repond pas au besoin initial

**Je m'interroge sur :**
- la possibilité de mettre une condition sur la présence dans le data du zonage EPCI ,
> genre if EPCI alors ajout du scroll ?

> oui c'était ça mon idée
> J'aime mieux la suivant, EPCI = TRUE dans le tableau peut servir à n'ajouter que qq epci en fonction du choix utilisateur à la création du projet
ou
- un nouvel argument pour laisser le choix de mettre le scroll ou non.

> j'aime beaucoup plus aussi cette option, car perso, je ne pense pas que j'utiliserais le scroll ;-)
On part la dessus, avec une vignette qui comprendra un exemple pour montrer aux Edito

>ok je m'en occupe, :+1: si
>pour l'exmeple, il faudra créer un dataset avec filter(indicateurs, Zone_ref | TypeZone == "Epci"):+1:

> **COMPLETER NEWS.Rd**


## Regrouper les lignes correspondants aux régions et France afin de pouvoir comparer le chiffre régional avec les taux France

OK

## Classer les EPCI par ordre alphabétique voire département

OK

# RETOUR:

## ajout d'un titre :

- en ajoutant le paramètre titre à la fonction `creer_tableau_4_1()`, apparition d'une nouvelle note: :cry:

:warning:
idem pour: `creer_graphe_2_1`, `creer_graphe_3_2`,` creer_graphe_4_1`, `creer_graphe_6_1` et `creer_graphe_6_2`.

> checking Rd line widths ... NOTE
  Rd file 'creer_tableau_4_1.Rd':
    \usage lines wider than 90 characters:
           titre = "Répartition des logements sociaux selon leur ancienneté<br>au 1er janvier {annee}",
  
  These lines will be truncated in the PDF manual.

Je ne trouve pas comment faire en sorte que le Rd soit modifié:

\usage{
creer_tableau_4_1(
  data,
  annee,
  epci = FALSE,
  add_scroll = FALSE,
 
    titre = "Répartition des logements sociaux selon leur ancienneté<br>au 1er jfceanvier {annee}",
  note_de_lecture = ""
)

- attention, dans les tableaux le saut de ligne se fait pas `<br>` alors que dans les cartes et graphes par `\n`

- problème de Warning pour la fonction non expliqué pour le moment`creer_graphe_2_1()`

W  checking for code/documentation mismatches (7.7s)
   Codoc mismatches from documentation object 'creer_graphe_2_1':
   creer_graphe_2_1
     Code: function(data, annee, palette = "pal_gouv_qual2", titre =
                    "Répartition {dep_reg}e des logements sociaux \nselon
                    le nombre de pièces au 01/01/{annee}", note_de_lecture
                    = "")
     Docs: function(data, annee, palette = "pal_gouv_qual2", titre =
                    "Répartition {dep_reg}e des logements sociaux \nselon
                    le nombre de pièces au 01/01/{annee}",
                    note_de_lecture = "")
     Incohérence dans les valeurs par défaut des arguments :
       Name: 'titre' Code: "Répartition {dep_reg}e des logements sociaux \nselon le nombre de pièces au 01/01/{annee}" Docs: "Répartition {dep_reg}e des logements sociaux \nselon le nombre de pièces au 01/01/{annee}"
   

# **Solution** :
Passer par un paramètre `titre=NULL` +

  if ( is.null(titre)){
    titre <-...}