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

Ajouter un onglet Éléments transverses #706

Closed
4 of 6 tasks
benoitdequick opened this issue May 16, 2024 · 10 comments · Fixed by #758
Closed
4 of 6 tasks

Ajouter un onglet Éléments transverses #706

benoitdequick opened this issue May 16, 2024 · 10 comments · Fixed by #758
Assignees
Labels
amélioration Proposition de nouvelle fonctionnalité ou amélioration de fonctionnalité Critères transverses

Comments

@benoitdequick
Copy link
Contributor

benoitdequick commented May 16, 2024

On remplace le toggle (#473 et #475) par un onglet dédié !

UX/UI

⚠️ Edit : voir à la fin du ticket

DEV

  • Supprimer le toggle “transverse”
  • Ajouter un nouvel onglet transverse (page spéciale renvoyée par le back)
  • Calculer le taux de conformité
  • Déprécier l’attribut “transverse”
  • Côté rapport, ne pas afficher en doublon les erreurs (section “Transverse” et section “Page X”)
  • [ ] Si un critère où “transverse = true” est édité, on passe transverse à false et on le remonte sur le rapport dans la section “Page X”.
  • Reprise des données des anciens audits (définir les règles)
@benoitdequick benoitdequick added amélioration Proposition de nouvelle fonctionnalité ou amélioration de fonctionnalité Critères transverses labels May 16, 2024
@benoitdequick benoitdequick added this to the Spring Summer 2 milestone May 16, 2024
@benoitdequick benoitdequick changed the title Appliquer le résultat d'un critère sur TOUTES les pages Ajouter un onglet Éléments transverses Jul 4, 2024
@bellangerq bellangerq self-assigned this Jul 5, 2024
@bellangerq
Copy link
Collaborator

@AdrienMuzyczka, dans les specs des onglets du DSFR, l'icône peut seulement se situer à gauche dans le bouton : https://www.systeme-de-design.gouv.fr/composants-et-modeles/composants/onglet.

Est-ce qu'on suit la reco ou on la met à droite ?

@AdrienMuzyczka
Copy link
Collaborator

@AdrienMuzyczka, dans les specs des onglets du DSFR, l'icône peut seulement se situer à gauche dans le bouton : https://www.systeme-de-design.gouv.fr/composants-et-modeles/composants/onglet.

Est-ce qu'on suit la reco ou on la met à droite ?

Pour le moment, je propose de ne rien toucher. Il se peut que l'icône repasse à gauche en fonction des résultats des tests et des itérations qui vont suivre. En revanche, si jamais on décide de garder l'icône à droite, il faudra customiser l'onglet du DSFR

@bellangerq
Copy link
Collaborator

@AdrienMuzyczka, si j'ai évalué le critère comme non applicable sur l'onglet "Éléments transverses", ça n'est pas cohérent puisque le message parle d""erreur". Il faudrait soit MAJ le wording en fonction du résultat (non-conforme ou non-applicable) ou mettre un wording générique pour les deux.

Image

@benoitdequick
Copy link
Contributor Author

benoitdequick commented Jul 10, 2024

Ça tombe très bien. On va modifier ce message et rappeler l’état déclaré dans l’onglet transverse.
Ex. "Vous avez déjà évalué ce critère à n/a sur toutes les pages"
(La formulation exacte n’est pas arrêtée)

@bellangerq
Copy link
Collaborator

bellangerq commented Jul 12, 2024

Quelques questions supplémentaires :

  1. Est-ce qu'on affiche la "page" "Éléments transverses" dans le tableau "Echantillon" de la page "Contexte" et dans le tableau "Répartition des critères par pages" l'onglet "Résultats" de la page "Rapport". Si oui, que met-on comme URL ?

Image

Image

  1. Est-ce qu'on garde le wording "Toutes les pages" côté rapport (dans le menu à gauche et dans le titre de la section transverse) ou on mets "Éléments transverses" ?

@yaaax
Copy link
Collaborator

yaaax commented Jul 12, 2024

Je dirais :
1)

  • Échantillon : non
  • Répartition des critères par pages : on ne met pas l'info dans le tableau existant mais à part ?
  1. "Éléments transverses"

@AdrienMuzyczka
Copy link
Collaborator

AdrienMuzyczka commented Jul 29, 2024

Toutes les maquettes de l'onglet "Éléments transverses"

Onglet transverse

  • Renommer l'onglet "Éléments transverse (optionnel)"
  • Placer l'icône à gauche dans l'onglet + remplacer par icône "Layout-5-line"
  • Supprimer la mention "Compléter les critères de cet onglet est optionnel" dans l'onglet
  • Supprimer le message d'information "Vous pouvez indiquer dans cet onglet les critères [...]"
  • Pour les critères dans cet onglet, renommer les accordéons :
    • "Erreur et recommandation" en "Erreur et recommandation sur toutes les pages"
    • "Commentaire" en "Commentaire sur toutes les pages"
    • "Point d'amélioration" en "Point d'amélioration sur toutes les pages" + enlever les "s" à "Points d'améliorations"
  • Ajouter le bouton radio "Conforme" aux critères

Page échantillon

  • Dans les blocs critères, enlever les "s" aux libellés des accordéons "Points d'améliorations"
  • Afficher un message dans les blocs "critères" quand le critère est évalué en transverse + un bouton permettant d'afficher en dessous la saisie qu'a effectué l'auditeur dans l'onglet transverse concernant ce critère.
    • Passer la couleur de l'icône "information" en noir
    • Le message reprend un tag (C, NC, NA) qui s'adapte en fonction du statut du critère en transverse
    • Le bouton s'affiche que si l'auditeur a réalisé une saisie dans le critère en transverse et le libellé s'adapte en fonction du statut du critère en transverse. La mise en forme des saisies de l'auditeur est gardée.
    • Cf maquette et prototype

Rapport d'audit

  • Dans la page "Contexte", ne pas afficher la page"Éléments transverses" dans le tableau "Échantillon"
  • Dans la page "Synthèse des résultats", tableau "Répartition des critères par pages"
    • Supprimer la page nommée "Éléments transverses". Les critères transverses doivent être comptés dans les pages de l'échantillon : si 1 NC concerne un élément transverse, la NC est comptée dans la page 1, 2, 3, 4 etc....
    • Ajouter une mise en avant "Éléments transverses" + lien redirigeant dans l'onglet "Détails des non-conformités". Cf maquette
  • Supprimer le "cadre" autour de tous les tableaux de la page. Cf maquette
  • Dans la page "Détails des non-conformités" les NC transverses sont affichées comme aujourd'hui dans une "page" nommée "Toutes les pages" (titre de la section + lien dans le menu de navigation à gauche).

Suite

Aura lieu à la rentrée et fait l'objet d'un autre ticket

@bellangerq
Copy link
Collaborator

Côté rapport, qu'en est-il du compte des critères conformes dans le tableau "Répartition des critères par thématiques du RGAA" ? Est-ce qu'on additionne les critères transverses dedans aussi ? J'imagine que oui vu qu'on l'a fait sur le tableau de répartition par page.

@AdrienMuzyczka
Copy link
Collaborator

AdrienMuzyczka commented Oct 2, 2024

Options de migration :

  1. On ne touche rien pour les anciens audits

Page"audit" : Pour les critères appliqués à toutes les pages avec l'interrupteur "Sur toutes les pages", le statut du critère est conservé et le commentaire est dupliqué sur toutes les pages. Par exemple, si le critère 1.1 a été évalué comme "Non Conforme" sur toutes les pages, ce critère reste "Non Conforme" sur toutes les pages et le commentaire est dupliqué sur chaque page. Même principe pour les critères appliqués à plusieurs pages.
Rapport : le statut du critère et le commentaire est répété dans toutes les pages si le toggle "sur toutes les pages" était coché. Même principe, s'il y avait une exception.

  • Avantages : facilite la migration
  • Inconvénients : dans le rapport les NC transverses sont répétées sur toutes les pages. Dans le cadre d'un contre-audit la modification d'un critère doit se faire sur toutes les pages pour un critère transverse.
  1. On déplace les critères identiques sur toutes les pages dans l'onglet transverse.
  • Avantages : transparent pour l'usager.
  • Inconvénients : migration complexe et chronophage. Peut être déroutant pour l'usager de voir ses saisies déplacées. Perte de saisie potentielle Si un critère transverse contenant des saisies propre à une page, le contenu propre a la page est déplacé dans l'onglet "éléments transverses".

### Règles pour la migration

Cas 1 : le toggle sur "toutes les pages" est activé sur toutes les pages (NC, NA, C)
- Comportement : Le contenu de l'accordéon (commentaire, impact, image, difficulté) est déplacé dans l'onglet "élément transverse". Sur les pages de l'échantillon, le statut du critère = statut élément transverse. Sur les autres pages le statut du critère reste identique.

Cas 2 : le toggle sur "toutes les pages" est activé sur plusieurs pages (mais pas toutes)
- On ne touche rien. Les pages restent identiques, seul le toggle "sur toutes les pages" est supprimée.

@AdrienMuzyczka
Copy link
Collaborator

AdrienMuzyczka commented Oct 16, 2024

Suite à la revue de design, dans l'onglet "Éléments transverses" :

  • Ajouter un message d'information dans les onglets des pages à auditer
    • Message affiché pour tous les usagers et sur tous les audits
    • Message présent dans tous les onglets de l’échantillon des pages à auditer
    • Si l’usager ferme une fois le message, il n’est plus jamais affiché
    • Message supprimé automatiquement dans 3 mois
    • Message ajouté dans les “notes de version”
  • (R)Ajouter le toggle permettant de masquer une thématique
    • (Re)Nommer ce toggle "Non applicable pour les éléments transverses"
  • Simplifier le libellé des accordéons "Erreur et recommandation sur toutes les pages" par "Erreur et recommandation"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amélioration Proposition de nouvelle fonctionnalité ou amélioration de fonctionnalité Critères transverses
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants