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

[explo-processus] Indiquer ce que serait la méthode de mise à jour avec la méthode {charpente} #9

Closed
9 of 13 tasks
statnmap opened this issue Jul 13, 2022 · 2 comments
Assignees

Comments

@statnmap
Copy link
Contributor

statnmap commented Jul 13, 2022

ETQ futur mainteneurs et utilisateurs, je souhaite savoir ce que cela implique d'intégrer une mise à jour esthétique du DSE dans une app shiny existante si la méthode est {charpente}, afin de choisir la méthode adaptée

Critères d'acceptation

  • Je sais pourquoi nous n'avons pas directement retenu la méthode {charpente}:
    Utiliser {charpente} implique de recoder la partie HTML d'une composante avec les fonctions de {shiny} type tags$p(), tags$a(), ...
    Or, le DSFR fournit le code HTML brut, que l'on peut copier en l'état depuis le sources du DSFR dans un htmlTemplate() sans que cela ne réduise les possibilité de modification des composantes.
    Ainsi, sans repasser pas {charpente}, nous simplifions l'ajout et la maintenance des composants.

Un rapport / vignette avec :

Comment technique

  • Définir comment pourraient être créé les HTML sur-mesure des composantes, et intégrer le CSS du DSE
  • Indiquer comment ce serait utliisé avec {charpente} pour être appliqué à une Shiny existante
  • Ecrire en toutes lettres, les étapes à réaliser pour la transformation d'une application Shiny existante déjà en prod
  • Ecrire en toutes lettres, les étapes à réaliser pour mettre à jour le package si le DSE change (nouvelles polices, nouvelles couleurs, nouveau logos, ...)

=> Ces informations permettront aux commanditaires de choisir la méthode adaptée à leur besoin et leurs critères de succès à savoir

  • Eviter d'avoir des fonctions spécifiques pour ne pas gêner le passage d'un template à un autre si possible
  • Le package doit être maintenable par les utilisateurs et peut être mis à jour rapidement s'il y a des mises à jour des fichiers css/js/polices/images du côté Design System de l'Etat
Cervangirard pushed a commit that referenced this issue Jul 19, 2022
Tags : doc

Pourquoi ?

- etre capable d'evaluer les avantages et inconvenients de chaque methode
- donner une recommandation pour le commanditaire

Quoi ?

- exploration de plusieurs pistes sur comment coder les composants DSFR pour les adapter a shiny
- ajout d'un tableau de comparaison

Tickets

ticket #7 #8 #9
Cervangirard pushed a commit that referenced this issue Jul 20, 2022
Tags : doc

Pourquoi ?

- Avoir une idée des temps estimés et de la faisabilité

Quoi ?

- ajout d'un fichier xlsx

Tickets

ticket #7 #8 #9
MurielleDelmotte added a commit that referenced this issue Jul 20, 2022
Tags : fix, doc

Pourquoi ?

- completer les inforamtions pour eclairer les differentes methodes

Quoi ?

- completion vignettes/recommandation-pour-l-implementation-de-dsfr
- correction orthographe vignettes/documentation-inventaire

Tickets

ticket #7 #8 #9
@statnmap statnmap changed the title [explo-processus] Indiquer ce que serait la méthode de mise à jour avec {charpente} [explo-processus] Indiquer ce que serait la méthode de mise à jour avec la méthode {charpente} Jul 20, 2022
@jengelaere
Copy link
Contributor

Vraiment besoin de m'attarder sur l'option C qui n'a pas été retenue ?
Sur l'option D, je ne coche pas 'ETQ mainteneur, Je sais comment créer et modifier les composantes sur-mesure et intégrer les nouveaux CSS/JS fournis par le DSE dans mon package pour l'option D'
J'ai bien lu la doc, mais elle n'est pas suffisante pour moi, je dois avoir trop de lacune sur html, css et js.

@statnmap
Copy link
Contributor Author

statnmap commented Sep 7, 2022

Ajout d'info avec le #58
Validé en réunion du 2022-09-07

@statnmap statnmap closed this as completed Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

3 participants