Transformation de la bannière jaune pour les rdv à renseigner #4907
Conversation
victormours
force-pushed
the
remove-yellow-banner
branch
from
7e384d8 to
5f73511
Compare
There was a problem hiding this comment.
👍 super initiative, ce flash faisait vraiment tâche dans l’UX.
Je n’ai pas de super suggestion pour améliorer, je trouve que c’est pas mal tel quel. On passe d’un flash très visible à un message peu visible. Je pense qu’on pourra améliorer plus tard, peut-être avec une customisation de couleur pour cette info, quitte à ne pas respecter le DSFR stricto sensu ?
| span.ml-1.alert-link = t(".unknown_past_rdv_count", count: agent.unknown_past_rdv_count ) | ||
| button.btn.rdv-status-unknown_past.float-right = t(".link_to_unknown_past_rdv") | ||
| / locals(organisation: nil) | ||
| - if current_agent.unknown_past_rdv_count > 0 && !current_agent.conseiller_numerique? # See https://github.com/betagouv/rdv-solidarites.fr/issues/2245 |
There was a problem hiding this comment.
suggestion tout à fait minime et absolument pas bloquante: on pourrait introduire un alias de unknown_past_rdv_count appelé rdv_a_renseigner_count pour aligner les API dans ce fichier.
(on pourrait aussi carrément renommer la colonne mais c’est beaucoup de travail)
| @@ -1,6 +1,5 @@ | |||
| class Admin::AgentAgendasController < AgentAuthController | |||
| def show | |||
| @hide_rdv_a_renseigner_in_main_layout = true | |||
There was a problem hiding this comment.
cool de supprimer ça 👍
| @@ -1,11 +1,6 @@ | |||
| - agent_already_busy = controller.action_name.in?(["new", "create", "edit", "update"]) # Un agent qui remplit un formulaire ne va pas interrompre son action pour renseigner des rdvs | |||
There was a problem hiding this comment.
Je pense aussi que ça n’est plus pertinent maintenant que le message est dans le header, ça serait même un peu perturbant de le voir apparaître - disparaître
| / locals(organisation: nil) | ||
| - if current_agent.unknown_past_rdv_count > 0 && !current_agent.conseiller_numerique? # See https://github.com/betagouv/rdv-solidarites.fr/issues/2245 | ||
| li.list-inline-item.fr-mr-2w | ||
| = link_to a_renseigner_admin_organisation_rdvs_path(organisation || current_agent.rdvs.a_renseigner.first.organisation_id), class: "btn text-primary-blue link-dsfr" do |
There was a problem hiding this comment.
note à moi-même : le cas sans orga passée en param se produit sur le layout config de territoire. Avant cette PR l’info des RDV à renseigner n’apparaissait pas dans cette interface.
Captures d'écran
Contexte
Cette PR est une exploration pour voir comment on peut remplacer la bannière jaune des rdv à renseigner par un composant moins invasif et stressant, mais qui incite quand même les agents à cliquer dessus.
Changement de composant
Une première piste est celle montrée dans les screenshots ci-dessus : la mettre dans le header (puisque ce lien est "multi-organisation", car il affiche les rdv à renseigner de toutes les organisations de l'agent, et est présent peu importe l'organisation dans laquelle l'agent est en train de naviguer.
La proposition ici reprend l'idée de l'en-tête avec accès rapides proposée par le dsfr : https://www.systeme-de-design.gouv.fr/composants-et-modeles/composants/en-tete. Tout le texte est un lien vers la page.
Je suis très preneur d'idées de variations sur le texte/l'icône/la couleur.
Changement du délai pour renseigner les rendez-vous
On voit sur les pages de stats deux instances que le taux de rdv renseignés est stable au bout d'une ou deux semaines. Ça veut dire que généralement les gens ne renseignent pas les rendez-vous de plus de 2 semaines (et ça se comprend, moi aussi j'aurais du mal à me souvenir si l'usager est venu ou non).
En plus, ça doit être décourageant d'avoir un vieux rdv non renseigné qui date de 2022, et au pire ça pousse à les renseigner à tort.
On propose donc ici de ne compter que les rdv de moins de 2 semaines.
Changement de l'affichage pendant que l'agent est sur un formulaire ?
Avec le fonctionnement bandeau, on arrêtait de l'afficher pendant que l'agent était sur un formulaire, mais je sais pas si ça reste pertinent ici. Je suis preneur de votre avis.