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

Améliorer la mise en page de la page détails des annotations #2490

Closed
Alexmarmelab opened this issue Feb 6, 2025 · 15 comments · Fixed by #2491
Closed

Améliorer la mise en page de la page détails des annotations #2490

Alexmarmelab opened this issue Feb 6, 2025 · 15 comments · Fixed by #2491
Assignees
Labels
Triage Nouveau problème ou besoin à étudier et à classer

Comments

@Alexmarmelab
Copy link
Collaborator

Alexmarmelab commented Feb 6, 2025

  • Rapprocher les valeurs des labels pour améliorer la compréhension
  • Ajouter une barre fixée en bas, placer le bouton "Sauvegarder" et ajouter un bouton "Annuler" pour une cohérence des interfaces
  • Ajouter un lien Mailto sur l'email du contributeur

Proposition validée sur Mattermost :
Image

@github-actions github-actions bot added the Triage Nouveau problème ou besoin à étudier et à classer label Feb 6, 2025
@Alexmarmelab Alexmarmelab changed the title Améliorer la mise en page de la page détails Améliorer la mise en page de la page détails des annotations Feb 6, 2025
@AnaelKremer
Copy link
Collaborator

la mise en page ne correspond pas à la maquette, est-ce normal ?

Image

@AnaelKremer AnaelKremer self-assigned this Feb 11, 2025
@Alexmarmelab
Copy link
Collaborator Author

@AnaelKremer effectivement, Jonathan a profité d'être sur la carte pour faire des améliorations. Il a posté une proposition sur Mattermost jeudi qui a été accepté. J'ai mis à jour la carte avec la proposition validée.

Parmi les modifications :

  • les informations du champ ne s'affichent plus sur 4 lignes avec les labels à gauche et les informations à droite mais sur 2 lignes de la même façon que sur les autres interfaces
  • la partie de droite occupe plus de place de façon à ce que les lignes de "Valeur d'origine" ne soient pas trop longues et difficile à lire

Si d'autres éléments sont à améliorer, les retours sont toujours les bienvenus.

@yannguillemin yannguillemin self-assigned this Feb 11, 2025
@AnaelKremer
Copy link
Collaborator

Ok @Alexmarmelab , alors ça me convient comme ça !

@yannguillemin
Copy link
Collaborator

@AnaelKremer : cela avait été acté sur Mattermost
Image
Sinon, cela me convient.
Néanmoins : Le champ "Commentaire interne" doit obligatoirement être renseigné quelque soit l'état de l'annotation, ne devrait-on pas plutôt se limiter à le rendre obligatoire en cas de statut "Refusée" ?

@AnaelKremer
Copy link
Collaborator

Pour le commentaire interne on en avait discuté, il sert à dire comment a été faite la modif, si un enrichissement est fait à partir d'un dictionnaire pour corriger des valeurs, ou des replace, on encore de façon manuelle.

Si quelqu'un doit prendre la main en admin sur un corpus déjà annoté et modifié en conséquence, on avait dit qu'il fallait impérativement expliquer comment des corrections ont été apportées.

@yannguillemin
Copy link
Collaborator

@AnaelKremer : je comprends bien pour les statuts " Validée" et "Refusée", mais pour "à traiter" et "en cours", cela me semble superflu de rendre une valeur arbitraire obligatoire.

@AnaelKremer
Copy link
Collaborator

ok, à voir si c'est simple de ne le rendre obligatoire que pour certains status

@Alexmarmelab
Copy link
Collaborator Author

C'est possible à faire mais ça ajoute une petite magouille,
A confirmer avec les prochains tests, s'il y a un besoin derrière

@yannguillemin
Copy link
Collaborator

yannguillemin commented Feb 11, 2025

@Alexmarmelab et @AnaelKremer : oui, ce n'est pas essentiel mais je le mentionne car ça a m'a fait penser à une demande au sujet de la valeur arbitraire obligatoire pour copier un graphique qui était un peu gênante ou contraignante.

@camilledesalabert
Copy link
Collaborator

Je rejoins le commentaire de Yann sur "en cours" et "à traiter". Si possible, faire en sorte que le commentaire interne ne soit pas obligatoire

Autres remarques :

  • Le bouton sauvegarder affiche une icône de disquette que l'on ait fait une modification ou non (comportement différent de ce qui se passe dans affichage => icône et petite animation si changement)
  • quand on a sauvegardé, la page du détail ne se referme pas, et on ne rebascule pas sur le tableau général des annotations (différent aussi du fonctionnement ailleurs dans Lodex) )=> est-ce voulu ?

@nantonot
Copy link
Collaborator

nantonot commented Feb 12, 2025

15.0.4 :
Remarque sur le statut "A traiter" : par défaut il n'y a pas de commentaire interne puisque c'est le statut généré à la création de l'annotation par le contributeur
Et dans ce cas, même si le "gestionnaire" regarde l'annotation, il n'a pas à "signer" ni à sauvegarder s'il ne fait rien sur cette annotation
A partir du moment où il a commencé à vérifier l'annotation, il choisit le statut "En cours", et dans ce cas, la "signature" est pour moi obligatoire ; de plus, cela ne prend pas de temps de mettre "à voir" dans le commentaire

@camilledesalabert
Copy link
Collaborator

Pour moi la signature devrait être obligatoire dans les autres cas. J'ai noté qu'elle était facultative, même en cas de refus ou validation, à la différence du commentaire

@nantonot
Copy link
Collaborator

nantonot commented Feb 12, 2025

Même remarque que Camille sur le bouton Sauvegarder (ne ferme pas la page détail de l'annotation) ; il faut recliquer sur l'onglet pour retourner à la liste des annotations

@Alexmarmelab
Copy link
Collaborator Author

Alexmarmelab commented Feb 13, 2025

Comme on s'éloigne du contexte de la carte, je crée de nouvelles cartes pour :

Il existe déjà une carte pour la fermeture de la page détails au clic sur le bouton Sauvegarder

On en parle au prochain point de conception

@yannguillemin
Copy link
Collaborator

Au vu des autres cartes créées, je suis ok avec la mise en page et valide la carte actuelle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triage Nouveau problème ou besoin à étudier et à classer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants