-
Notifications
You must be signed in to change notification settings - Fork 24
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
Refonte UX/UI - Carte du contrôle a posteriori #3848
Conversation
f51277b
to
e5fd99c
Compare
e5fd99c
to
ad4e1bd
Compare
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! Préciser comment accéder à la fonctionnalité pour le métier :
|
Merci @francoisfreitag pour les corrections de tests 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne suis pas convaincu par le fait que le collapse soit plié, il est facile de rater une action à faire (il faut déplier tous les collapses).
Aussi, dans l’état déplié, la différence entre « modifier le justificatif » et « ajouter un justificatif » est ténue, difficile d’identifier les résultats à ajouter. Aussi, pour chaque justificatif, il faut à nouveau déplier la section pour ajouter le suivant.
Dans la page de détail d’une auto-prescription en vue DDETS, une fois la campagne close, un encart vide apparaît :
Il reste des breadcrumbs sur la page calendrier, c’est voulu ?
itou/templates/siae_evaluations/institution_evaluated_job_application.html
Outdated
Show resolved
Hide resolved
a86ef01
to
bb7e28c
Compare
Oui, ça se défend. En l'état, c'est comme cela que Joffrey avait imaginé le truc, mais je proposerais a Antoine cette remarque pour une éventuelle évolution future
Pareil, il faudrait imaginer une manière de différencier ces actions/boutons. Je proposerais aussi à Antoine. Pour le collapse refermé, il faudrait peut être voir pour faire du dev afin de garder le collapse ouvert si retour d'upload du form. Mais dans une autre PR je pense, et aussi voir si tous ces "investissement" valent le coup, car la partie "controle a postériori" est sensée disparaitre l'année prochaine (si je ne me trompe pas)
Il y avait un breadcrumb et un retour. J'ai viré le retour, me disant qu'il reviendrait surement avec la bonne url et que le breadcrumb virerait avec la PR d'Antoine sur les remplacements des breadcrumb par les retours. |
Dans mes espoirs les plus fous, mais ça risque de prendre plus longtemps.
Je trouve que c’est une sacrée perte d’utilisabilité, je viens de demander un confirmation sur Slack : https://itou-inclusion.slack.com/archives/CQ6C3LSAH/p1713188682028809 |
bb7e28c
to
5adba29
Compare
c8dd50e
to
6625a17
Compare
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
6625a17
to
44a7609
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'avais déjà pas mal corrigé la mise en forme (la carte était dans le form, en col-6 etc...) et je ne préfère pas aller trop loin. Ces pages de form sont assez mal fichu niveau UI et je pense que ça mériterait plutôt un travail ux et que des rustines. Pour plus tard donc ... |
Pourquoi ?
Évolutions UI
Comment ?
Maj du templates de pages et des cartes