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

Refonte UX/UI - Carte du contrôle a posteriori #3848

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented Mar 28, 2024

Pourquoi ?

Évolutions UI

Comment ?

Maj du templates de pages et des cartes

@hellodeloo hellodeloo self-assigned this Mar 28, 2024
@hellodeloo hellodeloo changed the title WIP WIP: Refonte UX/UI - Carte du contrôle a posteriori Mar 28, 2024
@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch from f51277b to e5fd99c Compare April 3, 2024 09:25
Copy link

@hellodeloo hellodeloo changed the title WIP: Refonte UX/UI - Carte du contrôle a posteriori Refonte UX/UI - Carte du contrôle a posteriori Apr 3, 2024
@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch from e5fd99c to ad4e1bd Compare April 3, 2024 09:39
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 8, 2024
Copy link

github-actions bot commented Apr 8, 2024

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !


Préciser comment accéder à la fonctionnalité pour le métier :

@hellodeloo
Copy link
Contributor Author

Merci @francoisfreitag pour les corrections de tests 👍

@hellodeloo hellodeloo requested review from xavfernandez and a team April 8, 2024 16:10
Copy link
Contributor

@francoisfreitag francoisfreitag left a 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.
image


Dans la page de détail d’une auto-prescription en vue DDETS, une fois la campagne close, un encart vide apparaît :
image


Il reste des breadcrumbs sur la page calendrier, c’est voulu ?

itou/static/css/itou.css Show resolved Hide resolved
@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch from a86ef01 to bb7e28c Compare April 15, 2024 12:48
@hellodeloo
Copy link
Contributor Author

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).

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

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.

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 reste des breadcrumbs sur la page calendrier, c’est voulu ?

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.

@francoisfreitag
Copy link
Contributor

la partie "controle a postériori" est sensée disparaitre l'année prochaine (si je ne me trompe pas)

Dans mes espoirs les plus fous, mais ça risque de prendre plus longtemps.

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

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

@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch from bb7e28c to 5adba29 Compare April 22, 2024 16:28
@hellodeloo hellodeloo removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 22, 2024
@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch 2 times, most recently from c8dd50e to 6625a17 Compare April 26, 2024 12:49
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 26, 2024
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@hellodeloo hellodeloo force-pushed the deloo/update-siae-evaluations-cards branch from 6625a17 to 44a7609 Compare April 29, 2024 13:10
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tant qu’on est à retoucher ces écrans, peut-être élargir la zone de commentaire pour la DDETS ?

image

Autrement, c’est bien mieux ! Merci pour la mise à jour 😃

@hellodeloo
Copy link
Contributor Author

Tant qu’on est à retoucher ces écrans, peut-être élargir la zone de commentaire pour la DDETS ?

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 ...

@hellodeloo hellodeloo added this pull request to the merge queue Apr 29, 2024
Merged via the queue into master with commit 90c56cb Apr 29, 2024
11 checks passed
@hellodeloo hellodeloo deleted the deloo/update-siae-evaluations-cards branch April 29, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants