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

Segments - Modifier le détail du segment dans la fiche navire #4037

Closed
louptheron opened this issue Jan 20, 2025 · 4 comments · Fixed by #4045 or #4079
Closed

Segments - Modifier le détail du segment dans la fiche navire #4037

louptheron opened this issue Jan 20, 2025 · 4 comments · Fixed by #4045 or #4079
Assignees
Labels

Comments

@louptheron
Copy link
Collaborator

Il faudrait changer le description du segment suite à la refonte des segments 2025.

Description actuelle :
Image

@AdelineCelier
Copy link
Collaborator

@louptheron

Voici les écrans à jour (modulo les qques ajustements à faire sur les questions que je t'ai posé dans Mattermost) :

  • dans l'onglet "pêche" de la fiche navire : écran du détail de la marée du navire, et écran avec tooltip pour définition du segment
  • dans l'onglet "résumé" de la fiche navire : écran du détail de la marée du navire, et écran avec tooltip pour définition du segment

Pour les règles d'affichage ou non des infos, je dirais :

  • dans la description de la marée, ne pas mettre de ligne pour le critère s'il n'est pas pris en compte dans la def du segment
  • dans le tooltip, affficher tous critères et mettre "aucun/aucune" dans le cas où le critère n'est pas pris en compte
  • possible de gérer dynamiquement les pluriels des intitulés en Slate Gray ?

@louptheron
Copy link
Collaborator Author

Je fais ce ticket en deux temps, 1) sans custom tooltip (PR ci-dessus), puis 2) avec le tooltip custom et les bon styles.

@github-project-automation github-project-automation bot moved this from To do to To deploy in INT for review in Board MonitorFish Jan 24, 2025
@louptheron louptheron moved this from To deploy in INT for review to To review in INT in Board MonitorFish Jan 27, 2025
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Jan 27, 2025

@louptheron quand tu dis "sans custom tooltip", cela veut dire sans tooltip du tout ? C'est juste pour être sure que c'est voulu qu'il ne se passe rien quand on survole les tooltip.
Pour ce qui est des espèces cible tu penses que c'est bon, pas besoin de tester en inté sur un bateau concerné ?

Sinon juste qques petites remarques de formulation / UI :

  • on avait parlé de mettre au singulier ou au pluriel les libellés en fonction du nb de valeurs, c'est toujours good ? tu le fais dans la PR suivante ?
  • pour le "majorité d'espèces à bord", il faudrait mettre "démersales" au féminin pluriel pour l'accorder avec "espèces"
  • pour l'icône du tooltip ce serait bien d'harmoniser et utiliser la même partout, donc plutôt celle ci-dessous.
    Image
  • dans l'onglet "pêche", l'icône du tooltip et le nom du segment ne sont pas centrés (j'arrive pas trop à voir si c'est l'icône qui est trop haute, ou le texte du segment trop bas...ou un peu des deux)
    Image

Evidemment rien de bloquant pour la mep.

@louptheron louptheron moved this from To review in INT to To do in Board MonitorFish Jan 27, 2025
@louptheron
Copy link
Collaborator Author

louptheron commented Jan 28, 2025

Aussi,

  • Ne pas afficher une valeur null quand elle l'est
    Image

@louptheron louptheron reopened this Jan 28, 2025
@github-project-automation github-project-automation bot moved this from To do to In progress in Board MonitorFish Jan 28, 2025
@louptheron louptheron moved this from In progress to To do in Board MonitorFish Jan 28, 2025
@louptheron louptheron moved this from To do to In progress in Board MonitorFish Jan 30, 2025
@louptheron louptheron moved this from In progress to In Code Review in Board MonitorFish Feb 6, 2025
@github-project-automation github-project-automation bot moved this from In Code Review to To deploy in INT for review in Board MonitorFish Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment