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

(1) Pouvoir sélectionner la valeur à annoter via un sélecteur #2485

Closed
Alexmarmelab opened this issue Feb 5, 2025 · 7 comments · Fixed by #2492
Closed

(1) Pouvoir sélectionner la valeur à annoter via un sélecteur #2485

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

Comments

@Alexmarmelab
Copy link
Collaborator

Alexmarmelab commented Feb 5, 2025

Probleme

Le nombre d'icône est trop importante : une par section et une par valeur pour les listes

Solution

Retirer l'icône de l'annotation sur les valeurs
Au clic sur "Annoter" un champ, donner 2 propositions

  • Commenter la section
  • Commenter une valeur

Si clique sur "Commenter une valeur" et que le champ a plusieurs valeurs, ajouter une étape avec un sélecteur de valeur unique (autocomplete). Après cette étape, afficher l'étape du commentaire

Sinon, afficher directement l'étape du commentaire

Hors scope

  • Sélecteur multiple de valeur, la gestion est plus complexe dans l'admin

Comment tester ?

Tester les plusieurs formats de champ pour vérifier que le comportement corresponde à celui attendu

@github-actions github-actions bot added the Triage Nouveau problème ou besoin à étudier et à classer label Feb 5, 2025
@Alexmarmelab Alexmarmelab changed the title Pouvoir sélectionner la valeur à annoter via un sélecteur, en annotant le champ (1) Pouvoir sélectionner la valeur à annoter via un sélecteur, en annotant le champ Feb 5, 2025
@ThieryMichel ThieryMichel self-assigned this Feb 5, 2025
@Alexmarmelab Alexmarmelab changed the title (1) Pouvoir sélectionner la valeur à annoter via un sélecteur, en annotant le champ (1) Pouvoir sélectionner la valeur à annoter via un sélecteur Feb 5, 2025
@AnaelKremer
Copy link
Collaborator

Ca fonctionne, y compris avec des listes au format html et sur de très longues listes

Image

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

camilledesalabert commented Feb 11, 2025

Testé en 15.0.4
Fonctionnement attendu : ok

  • pas possible de commenter la section et la valeur sans faire 2 commentaires différents (plus fastidieux, mais plus simple pour la gestion)
  • attention, si liste à puces de valeurs créée via un format "Texte - balises HTML", on ne peut pas commenter les valeurs indépendamment les unes des autres ! Idem pour d'autres formats d'affichage, comme des tableaux.

Autres remarques/demandes :

  • Le libellé de l'icône parle d'annoter un champ mais la modale parle de section
  • pourquoi "commenter une valeur" quand la valeur est unique ?
  • Pourquoi "corriger xxxx" si on veut juste commenter ?
  • dans les commentaires de valeurs, rappeler la valeur dans la modale est très pratique dans la la plupart des cas pour faire un copier/coller si besoin. Dans certains cas, cependant, le format utilisé donne à l'affichage un résultat très différent de la valeur d'origine. Par exemple, dans l'image ci-dessous, le tableau est issu d'une routine. Pourrait-on dans le paramétrage des annotations prévoir un toggle supplémentaire pour dire si on veut que la valeur soit reprise dans la modale ou non.

Image

Proposition :

  • si on simplifie le tooltip du bouton annoter en précisant juste "ajouter un commentaire", on pourrait, une fois choisie section ou valeur, préciser "commenter la section XXX" ou "commenter la(es) valeur(s) YYY"

@yannguillemin
Copy link
Collaborator

Pour l'annotation des sous-ressources, le sélecteur de valeurs affiche l'uid au lieu de l'identifiant de la sous-ressource qui est normalement formaté en "URL -Lien interne".

Image

Sinon pour les longues listes à puces : l'accordéon s'affiche bien et est fonctionnel.

Image

@camilledesalabert
Copy link
Collaborator

  • Les commentaires sur les graphiques dans la page graphique ne proposent qu'un commentaire simple, mais s'ils sont copiés dans la page d'accueil, on peut commenter la section ou la valeur... à homogénéiser ?

@yannguillemin
Copy link
Collaborator

Chose étrange, le comportement vis-à-vis du menu d'annotation pour les graphiques en page d'accueil ne semble pas homogène.
Je vous donne ci-dessous 2 exemples différents de graphiques en page d'accueil qui ne fournissent pas le même menu.

Image

Image

@yannguillemin
Copy link
Collaborator

@camilledesalabert :et @Alexmarmelab Comme on s'éloigne du sujet de la carte, je transfère le problème dans la carte suivante placée dans "To discuss" : #2512.

@Alexmarmelab
Copy link
Collaborator Author

Alexmarmelab commented Feb 13, 2025

@camilledesalabert j'ai ajouté une carte à propos de "Corriger [URI]"
@yannguillemin j'ai créé une carte pour le cas des annotations sur les sous-ressources
Top pour la création de carte sur les graphiques

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.

5 participants