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

Signalements - Retouches UI #3183

Closed
5 tasks done
AdelineCelier opened this issue May 14, 2024 · 5 comments · Fixed by #3566, #3591, #3635 or #3644
Closed
5 tasks done

Signalements - Retouches UI #3183

AdelineCelier opened this issue May 14, 2024 · 5 comments · Fixed by #3566, #3591, #3635 or #3644
Assignees
Labels

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented May 14, 2024

Ce ticket regroupe plusieurs tickets UI sur les signalements : tickets #3022, #1757, #1684

Utilisation des composants de monitor-ui

  • Composants radio du formulaire : prendre les radio buttons de monitor-ui (cf. cet écran)
  • Utiliser les icônes de monitor-ui : alerte (cloche) / observation (jumelles) / edit simple (juste le crayon) / archiver / supprimer

Autres points

  • La marge à droite du formulaire n'est plus bonne :
    image

  • Quand il n'y a pas de signalement, mettre la phrase "pas de signalement ouvert sur ce navire" au-dessus du bouton et justifié à gauche (cf. cet écran)

  • Les 2 bandeaux rouges superposés ne sont pas très lisibles (bandeau alerte + bandeau signalement) : je propose de mettre le bandeau de suspicion d'infraction en MaximumRed 15%, et l'icône + le texte en MaximumRed.
    image

@louptheron louptheron moved this to Backlog in Board MonitorFish May 27, 2024
@louptheron louptheron moved this from Backlog to To do in Board MonitorFish Aug 19, 2024
@louptheron louptheron self-assigned this Aug 19, 2024
@louptheron louptheron moved this from To do to In Code Review in Board MonitorFish Aug 21, 2024
louptheron added a commit that referenced this issue Aug 29, 2024
## Linked issues

- Créations des types et aggrégats dans le backend plutôt que dans le
frontend
- Migration de use-cases JS en TS
- Resolve #3183
- Resolve #3271

----

- [ ] Tests E2E (Cypress)
@github-project-automation github-project-automation bot moved this from In Code Review to To deploy in INT for review in Board MonitorFish Aug 29, 2024
@louptheron louptheron moved this from To deploy in INT for review to To review in INT in Board MonitorFish Aug 29, 2024
@louptheron
Copy link
Collaborator

louptheron commented Aug 29, 2024

Bugs identifiés :

  • Le bouton "Afficher plus de signalements" ne fonctionne plus
  • Il est possible d'éditer un signalement de type Alerte, alors qu'il faudrait cacher l'icône d'édition

@louptheron louptheron reopened this Aug 29, 2024
@github-project-automation github-project-automation bot moved this from To review in INT to In progress in Board MonitorFish Aug 29, 2024
@github-project-automation github-project-automation bot moved this from In progress to To deploy in INT for review in Board MonitorFish Aug 29, 2024
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Aug 30, 2024

@louptheron je me permets 2 mini points supplémentaires, puisqu'on est proches de la perfection maintenant sur les signalements 😇

  • Possible de mettre le bandeau "ouvrir/éditer un signalement" en rouge 15% comme le reste du formulaire, quand le signalement est une infraction ?
    Image

    • Vu ensemble, implique de la refacto de code
  • Il semble que la marge blanche inférieure est un peu plus petite que les autres
    Image

@AdelineCelier AdelineCelier reopened this Aug 30, 2024
@github-project-automation github-project-automation bot moved this from To deploy in PROD to In progress in Board MonitorFish Aug 30, 2024
@louptheron
Copy link
Collaborator

louptheron commented Aug 30, 2024

Un autre retour :

  • On affiche "Dernière alerte le ..." sur les observations alors qu'on devrait juste afficher le ...

Screenshot from 2024-08-30 17-32-48

@github-project-automation github-project-automation bot moved this from In progress to To deploy in INT for review in Board MonitorFish Sep 12, 2024
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Sep 12, 2024

Les derniers points remontés sont bons, désolée je me permets d'en remonter un dernier : possible d'augmenter les marges entre les icônes à droite de la carte (pastille avec nb de signalements, éditer, supprimer, archiver) ?
Il faut à minima 8 px de marge entre le haut de la carte et la 1ère icône + entre chaque icône.
Voire je me demande s'il n'en faut pas 12 ou 16...tu crois qu'on peut faire un test en live ensemble ? Merci !

@louptheron louptheron moved this from In progress to In Code Review in Board MonitorFish Sep 13, 2024
louptheron added a commit that referenced this issue Sep 13, 2024
## Linked issues

- Resolve #3188
- Resolve #3183
- #3594
- Ajout de documentation dans le Makefile:
![Screenshot from 2024-09-13
11-27-53](https://github.com/user-attachments/assets/ec585dca-0432-4c6e-b468-b74623ff3bba)

----

- [ ] Tests E2E (Cypress)
@louptheron louptheron moved this from In Code Review to To review in INT in Board MonitorFish Sep 17, 2024
@AdelineCelier
Copy link
Collaborator Author

C'est good !

@louptheron louptheron moved this from To deploy in PROD to Done in Board MonitorFish Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment